BUG-2159: fix wrong javadocs
authorRobert Varga <rovarga@cisco.com>
Thu, 15 Jan 2015 21:36:14 +0000 (22:36 +0100)
committerRobert Varga <rovarga@cisco.com>
Fri, 16 Jan 2015 11:57:38 +0000 (12:57 +0100)
Java 8 javadoc generation is a lot stricter than previous versions. This
patch fixes the offenders.

Change-Id: I8a6d7f11a8fb38919d5fa34ab01c3a82849bfa8f
Signed-off-by: Robert Varga <rovarga@cisco.com>
12 files changed:
code-generator/binding-data-codec/src/main/java/org/opendaylight/yangtools/binding/data/codec/gen/impl/AbstractStreamWriterGenerator.java
code-generator/binding-data-codec/src/main/java/org/opendaylight/yangtools/binding/data/codec/gen/spi/StaticConstantDefinition.java
code-generator/binding-generator-impl/src/main/java/org/opendaylight/yangtools/sal/binding/generator/impl/BindingGeneratorImpl.java
code-generator/binding-generator-util/src/main/java/org/opendaylight/yangtools/binding/generator/util/Types.java
code-generator/binding-model-api/src/main/java/org/opendaylight/yangtools/sal/binding/model/api/CodeGenerator.java
code-generator/binding-model-api/src/main/java/org/opendaylight/yangtools/sal/binding/model/api/ParameterizedType.java
code-generator/binding-type-provider/src/main/java/org/opendaylight/yangtools/sal/binding/yang/types/GroupingDefinitionDependencySort.java
yang/yang-binding/src/main/java/org/opendaylight/yangtools/yang/binding/Augmentable.java
yang/yang-binding/src/main/java/org/opendaylight/yangtools/yang/binding/BindingStreamEventWriter.java
yang/yang-binding/src/main/java/org/opendaylight/yangtools/yang/binding/DataObject.java
yang/yang-binding/src/main/java/org/opendaylight/yangtools/yang/binding/InstanceIdentifier.java
yang/yang-binding/src/main/java/org/opendaylight/yangtools/yang/binding/util/BindingReflections.java

index 4ac9aa77236aa1c488d399053eb4455bdd168420..43333f6b9d2a1bcdd09220047f975e6eb2003b64 100644 (file)
@@ -291,7 +291,7 @@ abstract class AbstractStreamWriterGenerator extends AbstractGenerator implement
      * for all reachable binding objects.
      *
      * @param type Binding type of augmentation
-     * @param node Schema of augmentation
+     * @param schema Schema of augmentation
      * @return Source for augmentation node writer
      */
     protected abstract DataObjectSerializerSource generateSerializer(GeneratedType type, AugmentationSchema schema);
index 1688afd7c572a4487fcc70881619cec4421187a8..72ba33ba673ba81c192a61ba08d074c44eb4fef5 100644 (file)
@@ -18,7 +18,7 @@ import com.google.common.base.Preconditions;
  * <li>name - property name</li>
  * <li>type - Java type for property</li>
  * <li>value - value to which property should be initialized</li>
- *
+ * </ul>
  */
 public class StaticConstantDefinition {
 
index 81a92581835222d9f831e919f7f09cf1046c72e9..ee0eb2e40db83280e0110fcc1ea49796402688a2 100644 (file)
@@ -210,7 +210,7 @@ public class BindingGeneratorImpl implements BindingGenerator {
      *         <ul>
      *         <li>are generated from <code>context</code> schema nodes and</li>
      *         <li>are also part of some of the module in <code>modules</code>
-     *         set</li>.
+     *         set.</li>
      *         </ul>
      * @throws IllegalArgumentException
      *             <ul>
index 605c989bbad18f52111d62e9677299ee110711de..123287cf503249d1875d60f6da006ace646e9cee 100644 (file)
@@ -100,7 +100,7 @@ public final class Types {
 
     /**
      * Returns an instance of {@link ParameterizedType} describing the typed
-     * {@link Map}<K,V>
+     * {@link Map}&lt;K,V&gt;
      *
      * @param keyType
      *            Key Type
@@ -114,7 +114,7 @@ public final class Types {
 
     /**
      * Returns an instance of {@link ParameterizedType} describing the typed
-     * {@link Set}<V> with concrete type of value.
+     * {@link Set}&lt;V&gt; with concrete type of value.
      *
      * @param valueType
      *            Value Type
@@ -126,7 +126,7 @@ public final class Types {
 
     /**
      * Returns an instance of {@link ParameterizedType} describing the typed
-     * {@link List}<V> with concrete type of value.
+     * {@link List}&lt;V&gt; with concrete type of value.
      *
      * @param valueType
      *            Value Type
index d721a6a917f2a8d4668957b883a3be894bd6fba8..bde957b209870feffb1b94f509d0b3b45d4ee344 100644 (file)
@@ -19,7 +19,6 @@ public interface CodeGenerator {
      * @param type
      *            Input type to be processed
      * @return generated JAVA code
-     * @throws IOException
      */
     String generate(Type type);
 
index 8adabcec949371f2d0f7508d961cf7d79885731c..06b84987e2c68f99aaf04c10e07921254fe65e85 100644 (file)
@@ -8,7 +8,7 @@
 package org.opendaylight.yangtools.sal.binding.model.api;
 
 /**
- * Represents an instance of simple parametrized type such as List<String>.
+ * Represents an instance of simple parametrized type such as List&lt;String&gt;.
  *
  * The parametrized Type is designed to be used to store information of Java
  * Generic Type. The array of {@link #getActualTypeArguments()} holds
@@ -20,7 +20,7 @@ public interface ParameterizedType extends Type {
      * Returns array of Types that are defined for Parameterized Type.
      * <br>
      * (for example if ParameterizedType encapsulates java generic Map that
-     * specifies two parameters Map<K,V> and the K is java.lang.Integer and V
+     * specifies two parameters Map&lt;K,V&gt; and the K is java.lang.Integer and V
      * is defined as GeneratedType the array will contain two Types to store
      * the information of generic parameters.)
      *
index b697bf99121377d019080e166d98652283ab86e5..2657fcbaba44475fbf96a0d3bed196656384ba2e 100644 (file)
@@ -32,17 +32,17 @@ public class GroupingDefinitionDependencySort {
 
     /**
      * Sorts set <code>groupingDefinitions</code> according to the mutual
-     * dependencies.<br />
+     * dependencies.<br>
      *
      * Elements of <code>groupingDefinitions</code> are firstly transformed to
      * {@link org.opendaylight.yangtools.yang.parser.util.TopologicalSort.Node
      * Node} interfaces and then are sorted by
      * {@link org.opendaylight.yangtools.yang.parser.util.TopologicalSort#sort(Set)
-     * sort()} method of <code>TopologicalSort</code>.<br />
-     * <br />
+     * sort()} method of <code>TopologicalSort</code>.<br>
+     * <br>
      *
      *
-     * <i>Definition of dependency relation:<br />
+     * <i>Definition of dependency relation:<br>
      * The first <code>GroupingDefinition</code> object (in this context)
      * depends on second <code>GroupingDefinition</code> object if the first one
      * contains in its set of <code>UsesNode</code> (obtained through
index c562a77d4fdc75bb4cd49263aa694bc6804df3db..832cdcdca3d8d1f3c5d38453e110f99225482cda 100644 (file)
@@ -16,7 +16,7 @@ package org.opendaylight.yangtools.yang.binding;
  * also adds marker interface for augmentations (extensions) -
  * {@link Augmentable}
  * 
- * @author Tony Tkacik <ttkacik@cisco.com>
+ * @author Tony Tkacik
  * @param <T>
  *            Base class which should implements this interface and is target
  *            for augmentation.
index 97b42f37ce53d8256600d5ef2602fe03013278ba..233af3ef291f6b3fcaf0e694c5219b1472667f86 100644 (file)
@@ -26,7 +26,7 @@ import java.io.IOException;
  *
  * <li><code>list</code> - YANG list statement has two representation in event
  * stream - unkeyed list and map. Unkeyed list is YANG list which did not
- * specify key.</li>
+ * specify key.
  *
  * <ul>
  * <li><code>Map</code> - Map start event is emitted using
@@ -37,8 +37,8 @@ import java.io.IOException;
  * <li><code>UnkeyedList</code> - Unkeyed list represent list without keys,
  * unkeyed list start is emitted using {@link #startUnkeyedList(Class, int)} list
  * end is emitted using {@link #endNode()}. Each list item is emitted using
- * {@link #startUnkeyedListItem()} and ended using {@link #endNode()}.</li>
- * </ul>
+ * {@link #startUnkeyedListItem(int)} and ended using {@link #endNode()}.</li>
+ * </ul></li>
  *
  * <li><code>leaf</code> - Leaf node event is emitted using
  * {@link #leafNode(String, Object)}. {@link #endNode()} MUST be not emitted for
@@ -47,7 +47,7 @@ import java.io.IOException;
  * <li><code>leaf-list</code> - Leaf list start is emitted using
  * {@link #startLeafSet(String, int)}. Leaf list end is emitted using
  * {@link #endNode()}. Leaf list entries are emitted using
- * {@link #leafSetEntryNode(Object).
+ * {@link #leafSetEntryNode(Object)}.
  *
  * <li><code>anyxml - Anyxml node event is emitted using
  * {@link #leafNode(String, Object)}. {@link #endNode()} MUST be not emitted
@@ -207,7 +207,7 @@ public interface BindingStreamEventWriter extends Closeable, Flushable {
      *
      * <p>
      * End of unkeyed list event is emitted by invoking {@link #endNode()}.
-     * Valid subevents is only {@link #startUnkeyedListItem()}. All other
+     * Valid subevents is only {@link #startUnkeyedListItem(int)}. All other
      * methods will throw {@link IllegalArgumentException}.
      *
      * @param localName
@@ -318,8 +318,6 @@ public interface BindingStreamEventWriter extends Closeable, Flushable {
      *
      * <p>
      * Valid sub-events are:
-     * <<p>
-     * Valid sub-events are:
      * <ul>
      * <li>{@link #leafNode(String, Object)}</li>
      * <li>{@link #startContainerNode(Class, int)}</li>
@@ -349,13 +347,11 @@ public interface BindingStreamEventWriter extends Closeable, Flushable {
      * Emits start of choice node.
      *
      * <p>
-     * Valid sub-event in {@link #startCase(QName, int)}, which selects case
+     * Valid sub-event in {@link #startCase(Class, int)}, which selects case
      * which should be written.
-     * <ul>
      *
-     * @param localName
-     *            name of node as defined in schema, namespace and revision are
-     *            derived from parent node.
+     * @param choice
+     *            Choice class.
      * @param childSizeHint
      *            Non-negative count of expected direct child nodes or
      *            {@link #UNKNOWN_SIZE} if count is unknown. This is only hint
@@ -363,8 +359,8 @@ public interface BindingStreamEventWriter extends Closeable, Flushable {
      *            events than count.
      * @throws IllegalArgumentException
      * @throws IllegalStateException
-     *             If node was emitted inside <code>map</code>, <code>choice
-     *             </code> <code>unkeyed list</code> node.
+     *             If node was emitted inside <code>map</code>, <code>choice</code>,
+     *             <code>unkeyed list</code> node.
      * @throws IOException if an underlying IO error occurs
      */
     void startChoiceNode(Class<? extends DataContainer> choice, int childSizeHint) throws IOException, IllegalArgumentException;
@@ -385,7 +381,7 @@ public interface BindingStreamEventWriter extends Closeable, Flushable {
      * <li>{@link #startAugmentationNode(Class)}</li>
      * </ul>
      *
-     * @param name
+     * @param caze Case class
      * @throws IllegalArgumentException
      * @throws IOException if an underlying IO error occurs
      */
@@ -400,8 +396,6 @@ public interface BindingStreamEventWriter extends Closeable, Flushable {
      * <p>
      * Valid sub-events are:
      *
-     * <p>
-     * Valid sub-events are:
      * <ul>
      * <li>{@link #leafNode(String, Object)}</li>
      * <li>{@link #startContainerNode(Class, int)}</li>
@@ -415,10 +409,7 @@ public interface BindingStreamEventWriter extends Closeable, Flushable {
      * Note this is only method, which does not require childSizeHint, since
      * maximum value is always size of <code>possibleChildren</code>.
      *
-     * @param module
-     *            QName module of YANG module in which augmentation was defined
-     * @param possibleChildren
-     *            Local names of all valid children defined by augmentation.
+     * @param augmentationType augmentation class
      * @throws IllegalArgumentException
      *             If augmentation is invalid in current context.
      * @throws IOException if an underlying IO error occurs
index b0ca831500b3351ee37a92eab9e50716a9d157f6..6d36646165ee0f4aac76f2317a6244389b682ac4 100644 (file)
@@ -10,8 +10,7 @@ package org.opendaylight.yangtools.yang.binding;
 /**
  * Data container is an interface which has structured contents.
  * 
- * 
- * @author Tony Tkacik <ttkacik@cisco.com>
+ * @author Tony Tkacik
  *
  */
 public interface DataObject extends DataContainer {
index 087a92c64b91f3de084af846306b6717ccca0bc0..445e2e6aaad6fb58ab5a125ddb3ac2ebe57d9189 100644 (file)
@@ -107,7 +107,7 @@ public class InstanceIdentifier<T extends DataObject> implements Path<InstanceId
      * Check whether an instance identifier contains any wildcards. A wildcard
      * is an path argument which has a null key.
      *
-     * @return @true if any of the path arguments has a null key.
+     * @return true if any of the path arguments has a null key.
      */
     public final boolean isWildcarded() {
         return wildcarded;
@@ -164,7 +164,7 @@ public class InstanceIdentifier<T extends DataObject> implements Path<InstanceId
      * quick checks on their specific fields.
      *
      * @param other The other identifier, guaranteed to be the same class
-     * @return @true if the other identifier cannot be equal to this one.
+     * @return true if the other identifier cannot be equal to this one.
      */
     protected boolean fastNonEqual(final InstanceIdentifier<?> other) {
         return false;
@@ -286,7 +286,7 @@ public class InstanceIdentifier<T extends DataObject> implements Path<InstanceId
      * non-wildcarded PathArgument counterpart.
      *
      * @param other Identifier which should be checked for inclusion.
-     * @return @true if this identifier contains the other object
+     * @return true if this identifier contains the other object
      */
     public final boolean containsWildcarded(final InstanceIdentifier<?> other) {
         Preconditions.checkNotNull(other, "other should not be null");
@@ -364,7 +364,7 @@ public class InstanceIdentifier<T extends DataObject> implements Path<InstanceId
     /**
      * Create a new InstanceIdentifierBuilder given a base InstanceIdentifier
      *
-     * @param basePath
+     * @param base
      * @param <T>
      * @return
      *
@@ -438,7 +438,7 @@ public class InstanceIdentifier<T extends DataObject> implements Path<InstanceId
      *
      * Example
      * <pre>
-     *  List<PathArgument> path = Arrays.asList(new Item(Nodes.class))
+     *  List&lt;PathArgument&gt; path = Arrays.asList(new Item(Nodes.class))
      *  new InstanceIdentifier(path);
      * </pre>
      *
index c8dad79e44ce21faac36444d3de24c022039a084..6d6decb8ec74d5bb25fa98e8b727f4306cc9b3dd 100644 (file)
@@ -88,11 +88,9 @@ public class BindingReflections {
      * This method uses first generic argument of
      * implemented {@link ChildOf} interface.
      *
-     * @param augmentation
-     *            {@link Augmentation} subclass for which we want to determine
-     *            augmentation target.
-     * @return Augmentation target - class which augmentation provides
-     *         additional extensions.
+     * @param childClass
+     *            child class for which we want to find the parent class.
+     * @return Parent class, e.g. class of which the childClass is ChildOf.
      */
     public static Class<?> findHierarchicalParent(final Class<? extends ChildOf<?>> childClass) {
         return ClassLoaderUtils.findFirstGenericArgument(childClass, ChildOf.class);
@@ -104,11 +102,9 @@ public class BindingReflections {
      * This method is shorthand which gets DataObject class by invoking
      * {@link DataObject#getImplementedInterface()} and uses {@link #findHierarchicalParent(Class)}.
      *
-     * @param childClass
-     *            {@link Augmentation} subclass for which we want to determine
-     *            augmentation target.
-     * @return Augmentation target - class which augmentation provides
-     *         additional extensions.
+     * @param child
+     *            Child object for which the parent needs to be located.
+     * @return Parent class, or null if a parent is not found.
      */
     public static Class<?> findHierarchicalParent(final DataObject child) {
         if (child instanceof ChildOf) {
@@ -229,7 +225,7 @@ public class BindingReflections {
     /**
      * Returns root package name for supplied package name.
      *
-     * @param pkg Package for which find model root package.
+     * @param name Package for which find model root package.
      * @return Package of model root.
      */
     public static String getModelRootPackageName(final String name) {