Fix javadoc JDK8 compatibility 13/19613/2
authorRobert Varga <rovarga@cisco.com>
Tue, 5 May 2015 12:39:30 +0000 (14:39 +0200)
committerMartin Sunal <msunal@cisco.com>
Tue, 12 May 2015 18:08:44 +0000 (18:08 +0000)
JDK8 is more picky about javadoc format, so fix them up to pass
compilation with JDK8.

Change-Id: Ib79d332135f4a29f98a1461a17ab86118d1a2678
Signed-off-by: Robert Varga <rovarga@cisco.com>
16 files changed:
groupbasedpolicy/src/main/java/org/opendaylight/groupbasedpolicy/endpoint/EndpointRpcRegistry.java
groupbasedpolicy/src/main/java/org/opendaylight/groupbasedpolicy/resolver/MatcherUtils.java
groupbasedpolicy/src/main/java/org/opendaylight/groupbasedpolicy/resolver/PolicyListener.java
groupbasedpolicy/src/main/java/org/opendaylight/groupbasedpolicy/resolver/TenantUtils.java
groupbasedpolicy/src/main/java/org/opendaylight/groupbasedpolicy/util/SingletonTask.java
renderers/ofoverlay/src/main/java/org/opendaylight/groupbasedpolicy/integration/openstackgbp/OpenstackGbpEndpoint.java
renderers/ofoverlay/src/main/java/org/opendaylight/groupbasedpolicy/renderer/ofoverlay/EndpointManager.java
renderers/ofoverlay/src/main/java/org/opendaylight/groupbasedpolicy/renderer/ofoverlay/flow/FlowUtils.java
renderers/ofoverlay/src/main/java/org/opendaylight/groupbasedpolicy/renderer/ofoverlay/flow/OfTable.java
renderers/ofoverlay/src/main/java/org/opendaylight/groupbasedpolicy/renderer/ofoverlay/flow/OrdinalFactory.java
renderers/ofoverlay/src/main/java/org/opendaylight/groupbasedpolicy/renderer/ofoverlay/sf/Classifier.java
renderers/opflex/src/main/java/org/opendaylight/groupbasedpolicy/renderer/opflex/EprContext.java
renderers/opflex/src/main/java/org/opendaylight/groupbasedpolicy/renderer/opflex/jsonrpc/ConnectionService.java
renderers/opflex/src/main/java/org/opendaylight/groupbasedpolicy/renderer/opflex/jsonrpc/JsonRpcEndpoint.java
renderers/opflex/src/main/java/org/opendaylight/groupbasedpolicy/renderer/opflex/jsonrpc/RpcBroker.java
renderers/opflex/src/main/java/org/opendaylight/groupbasedpolicy/renderer/opflex/mit/PolicyClassInfo.java

index bef2c5c30f1036c2bcc9eda099feef0f996d56ad..9b7e77e986de88b74259b9dd99af572be61bf310 100644 (file)
@@ -85,8 +85,6 @@ public class EndpointRpcRegistry implements EndpointService {
      *            - the dataProvider
      * @param rpcRegistry
      *            - the rpcRegistry
-     * @param executor
-     *            - thread pool executor
      * @param epRendererAugmentation
      *            - specific implementation RPC augmentation, if any. Otherwise
      *            NULL
index cf5cea330d01b96cd076007c755241cddd1b949d..bd2ce116a2fc13df2c59488bef312ff7cd3bf72d 100644 (file)
@@ -76,7 +76,7 @@ public class MatcherUtils {
      *
      * @param matcher
      *            the {@link RequirementMatcher} to apply
-     * @param target
+     * @param relator
      *            the {@link ConsumerSelectionRelator} to match against
      * @return <code>true</code> if the matcher matches the target
      */
@@ -102,7 +102,7 @@ public class MatcherUtils {
      *
      * @param matcher
      *            the {@link RequirementMatcher} to apply
-     * @param target
+     * @param relator
      *            the {@link ProviderSelectionRelator} to match against
      * @return <code>true</code> if the matcher matches the target
      */
index 5e8c71981a9258e2d086642b68219fbb721427d9..2d1afe83ace24eef1b05b4c781ece3b4c4926cba 100644 (file)
@@ -20,7 +20,7 @@ public interface PolicyListener {
     /**
      * Indicates that the policy related to the set of endpoint groups
      * has changed
-     * @param updatedConsumers the set of endpoint groups involved
+     * @param updatedGroups the set of endpoint groups involved
      * in a contract whose policy has changed
      */
     public void policyUpdated(Set<EgKey> updatedGroups);
index 159d170c4bf9217011666b1c89973f661b8868a9..f425b5867bce710b61097a341dfd4b30b0a3def6 100644 (file)
@@ -104,8 +104,8 @@ public class TenantUtils {
 
     /**
      * Find a clause with a specified name within a contract
-     * @param tenant the {@link Contract} to search
-     * @param contractId the {@link ClauseName} to search for
+     * @param contract the {@link Contract} to search
+     * @param clauseName the {@link ClauseName} to search for
      * @return the {@link Clause} if it exists, null otherwise
      */
     public static Clause findClause(Contract contract, 
@@ -122,7 +122,7 @@ public class TenantUtils {
 
     /**
      * Find a subject with a specified name within a contract
-     * @param tenant the {@link Contract} to search
+     * @param contract the {@link Contract} to search
      * @param subjectName the {@link SubjectName} to search for
      * @return the {@link Subject} if it exists, null otherwise
      */
@@ -177,7 +177,7 @@ public class TenantUtils {
     /**
      * Find a consumer named selector in an endpoint group
      * @param eg the {@link EndpointGroup} to search
-     * @param name the {@link NamedSelectorName} to search for
+     * @param name the {@link SelectorName} to search for
      * @return the {@link ConsumerNamedSelector} if it exists, null otherwise
      */
     public static ConsumerNamedSelector findCns(EndpointGroup eg,
@@ -211,7 +211,7 @@ public class TenantUtils {
     /**
      * Find a provider named selector in an endpoint group
      * @param eg the {@link EndpointGroup} to search
-     * @param name the {@link NamedSelectorName} to search for
+     * @param name the {@link SelectorName} to search for
      * @return the {@link ProviderNamedSelector} if it exists, null otherwise
      */
     public static ProviderNamedSelector findPns(EndpointGroup eg,
@@ -228,7 +228,7 @@ public class TenantUtils {
     /**
      * Find a provider target selector in an endpoint group
      * @param eg the {@link EndpointGroup} to search
-     * @param name the {@link TargetSelectorName} to search for
+     * @param name the {@link SelectorName} to search for
      * @return the {@link ProviderNamedSelector} if it exists, null otherwise
      */
     public static ProviderTargetSelector findPts(EndpointGroup eg,
index 3669048b29085e433566b24ecf8f4de49c34c187..053265079308273b4a3b55cd4d59679b97fb5b47 100644 (file)
@@ -88,8 +88,8 @@ public class SingletonTask {
      * Construct a new SingletonTask for the given runnable.  The context
      * is used to manage the state of the task execution and can be shared
      * by more than one instance of the runnable.
-     * @param context
-     * @param Task
+     * @param ses
+     * @param task
      */
     public SingletonTask(ScheduledExecutorService ses,
             Runnable task) {
@@ -146,4 +146,4 @@ public class SingletonTask {
                 ses.schedule(stw, delay, unit);
         }
     }
-}
\ No newline at end of file
+}
index f3f76f35d7f3f7e16fe9032b79a9be5b4bae6e5a..ad03160e1e99b96518a122d7b03b5a2821692f77 100644 (file)
@@ -148,8 +148,6 @@ public class OpenstackGbpEndpoint implements AutoCloseable,
      *
      * @param input
      *            the input object
-     * @param timestamp
-     *            the current timestamp
      */
     protected EndpointBuilder buildEndpoint(RegisterEndpointInput input) {
         return new EndpointBuilder(input);
@@ -161,8 +159,6 @@ public class OpenstackGbpEndpoint implements AutoCloseable,
      *
      * @param input
      *            the input object
-     * @param timestamp
-     *            the current timestamp
      */
     protected EndpointL3Builder buildEndpointL3(RegisterEndpointInput input) {
         return new EndpointL3Builder(input);
index 9cb8ffc7360eb53e69c0133be4a9643c140e8d61..551f6e7278eebfd998251e117bd70b54a21e7389 100644 (file)
@@ -244,8 +244,8 @@ public class EndpointManager implements AutoCloseable, DataChangeListener
     /**
      * Get a collection of endpoints in a particular endpoint group
      *
-     * @param nodeId
-     *            the nodeId of the switch to get endpoints for
+     * @param eg
+     *            endpoint group ID
      * @return a collection of {@link Endpoint} objects.
      */
     public synchronized Collection<Endpoint> getEndpointsForGroup(EgKey eg) {
index c92a697d3921354fa7dd0657c9f6d0c1e9a8e4fe..8b9a3fbea08b61c486076a8ab3e66a1b5652fe02 100644 (file)
@@ -164,7 +164,7 @@ public final class FlowUtils {
      *
      * @param nodeId the ID of the node
      * @param tableId the ID of the table
-     * @return the {@link InstanceIdentifier<Table>}
+     * @return the {@link InstanceIdentifier}
      */
     public static final InstanceIdentifier<Table> createTablePath(final NodeId nodeId, final short tableId) {
         return createNodePath(nodeId).builder()
@@ -178,7 +178,7 @@ public final class FlowUtils {
      *
      * @param nodeId the Id of the node
      * @param groupId the ID of the group table
-     * @return the {@link InstanceIdentifier<Group>}
+     * @return the {@link InstanceIdentifier}
      */
     public static final InstanceIdentifier<Group> createGroupPath(final NodeId nodeId, final GroupId groupId) {
         return createNodePath(nodeId).builder()
@@ -193,7 +193,7 @@ public final class FlowUtils {
      * @param nodeId the Id of the node
      * @param groupId the ID of the group table
      * @param bucketId the ID of the bucket in the group table
-     * @return the {@link InstanceIdentifier<Bucket>}
+     * @return the {@link InstanceIdentifier}
      */
     public static final InstanceIdentifier<Bucket> createBucketPath(final NodeId nodeId, final GroupId groupId,
             final BucketId bucketId) {
index 53f808d4047f1ec812ce4cbbbe2ec804ccc6b620..a2ccc1acb4b1d8a215d6c75cbca405c4b424f6f2 100644 (file)
@@ -40,7 +40,6 @@ public abstract class OfTable {
     /**
      * Update the relevant flow table for the node
      * @param nodeId the node to update
-     * @param dirty the dirty set
      * @param policyInfo
      * @param flowMap
      * @throws Exception
index f9f1aad0fcaccac1a5306712d9a917056543d0d5..5f3225dbc4f8255007dac5fc690d5d4da57d2f51 100644 (file)
@@ -79,10 +79,8 @@ public class OrdinalFactory {
      * Get a 32-bit context ordinal suitable for use in the OF data plane for
      * the given policy item.
      *
-     * @param tenantId
-     *        the tenant ID of the element
-     * @param id
-     *        the unique ID for the element
+     * @param destNode
+     *        destination node ID
      * @return the 32-bit ordinal value
      * @throws Exception
      */
index bf59d95fdeeff3438405abb0fa3ba7988a7a39d8..cb5717f6d6e273b2f1e855b8051a63fefd87e79b 100644 (file)
@@ -50,8 +50,7 @@ public abstract class Classifier {
     public abstract ClassifierDefinition getClassDef();
 
     /**
-     * @return parent classifier
-     * @see {@link ClassifierDefinition}
+     * @return parent classifier, see {@link Classifier}
      */
     public final Classifier getParent() {
         return parent;
@@ -63,8 +62,7 @@ public abstract class Classifier {
      * @param matches list of builders containing {@code matches} to update
      * @param params parameters of classifier-instance inserted by user
      * @return result, which indicates if all the matching fields were updated successfully and
-     *         contain updated {@code matches}
-     * @see {@link ClassificationResult}
+     *         contain updated {@code matches}, see {@link ClassificationResult}
      */
     public final ClassificationResult updateMatch(List<MatchBuilder> matches, Map<String, ParameterValue> params) {
         if (params == null) {
@@ -103,13 +101,11 @@ public abstract class Classifier {
         return new ClassificationResult(matchBuilders);
     }
 
-/**
+    /**
      * Checks presence of required {@code params} in order to decide if classifier can update {@code matches} properly
-     * in  method {@link #update(List, Map)
+     * in  method {@link #update(List, Map)}
      * @param  params  inserted parameters, not null
-     * @return  true, if required parameters are present
-     * @throws  IllegalArgumentException when any of required {@code params} is not present
-     * @see  {@link #updateMatch(List, Map)}
+     * @throws  IllegalArgumentException when any of required {@code params} is not present, see {@link #updateMatch(List, Map)}
      */
     protected abstract void checkPresenceOfRequiredParams(Map<String, ParameterValue> params);
 
index ea95984eacb136b5e35e4373b57a6b8a402f4be9..1f061070cf0ccb49da85b232da475599b7152052 100644 (file)
@@ -89,7 +89,7 @@ public class EprContext implements Runnable, EprOperation.EprOpCallback {
 
     /**
      * Create an Endpoint in the Endopint Registry. This can only be called in
-     * response to an {@link EndpointDeclareRequest} message
+     * response to an {@link org.opendaylight.groupbasedpolicy.renderer.opflex.lib.messages.EndpointDeclareRequest} message
      */
     public void createEp() {
 
@@ -108,7 +108,7 @@ public class EprContext implements Runnable, EprOperation.EprOpCallback {
 
     /**
      * Delete an Endpoint in the Endpoint Registry. This can only be called in
-     * response to an {@link EndpointUndeclareRequest} message
+     * response to an {@link org.opendaylight.groupbasedpolicy.renderer.opflex.lib.messages.EndpointUndeclareRequest} message
      */
     public void deleteEp() {
 
@@ -126,7 +126,7 @@ public class EprContext implements Runnable, EprOperation.EprOpCallback {
 
     /**
      * Look up an endpoint in the Endpoint Registry. This can only be called in
-     * response to an {@link EndpointResolveRequest} message. It initiates all
+     * response to an {@link org.opendaylight.groupbasedpolicy.renderer.opflex.lib.messages.EndpointResolveRequest} message. It initiates all
      * of the reads, one by one, and invokes the callback when all of them have
      * completed.
      */
index 502d763c10a5ab469dbcb9f8c52037352dc595b3..1a7468d49533f9b00a530b921ae630d26c2d96a0 100644 (file)
@@ -13,8 +13,8 @@ package org.opendaylight.groupbasedpolicy.renderer.opflex.jsonrpc;
 /**
  * An interface to provide notifications when connections are
  * established or closed. The connection notifications
- * use{@link RpcEncpoint} objects; as connections come and go,
- * the {@link RpcEndpoint} objects associated with the connections
+ * use {@link JsonRpcEndpoint} objects; as connections come and go,
+ * the {@link JsonRpcEndpoint} objects associated with the connections
  * can be long-lived
  *
  * @author tbachman
index 80e410688be6393f818a2bb1428abd82c089e9a2..5fd3c84a9d0ddf7804bf238e65cd9d50b3ee7e5e 100644 (file)
@@ -103,7 +103,7 @@ public class JsonRpcEndpoint implements ChannelFutureListener {
      * Send a concrete {@link RpcMessage} to the RPC endpoint.
      *
      * @param message The concrete {@link RpcMessage} to send
-     * @return SettableFuture<Object> The caller can use the returned
+     * @return SettableFuture&lt;Object&gt; The caller can use the returned
      * object to wait for the response (currently no timeout)
      * @throws Throwable The concrete message couldn't be serialized and sent
      */
index cbd468e0a0a54c8fe58fb885fc7f072d3e1bc044..95ccf7d3691c9bac268b8a9821372fc5a0d71e7c 100644 (file)
@@ -33,7 +33,7 @@ public interface RpcBroker {
          * request message is received
          *
          * @param endpoint The endpoint that received the messgae
-         * @param message The concrete {@RpcMessage} received
+         * @param message The concrete {@link RpcMessage} received
          */
         public void callback(JsonRpcEndpoint endpoint, RpcMessage message);
 
@@ -41,7 +41,7 @@ public interface RpcBroker {
 
     /**
      *
-     * Subscribe to a concrete {@RpcMessage}
+     * Subscribe to a concrete {@link RpcMessage}
      *
      * @param message The concrete {@link RpcMessage} message to subscribe to
      * @param callback The callback to invoke when the message is published
@@ -54,7 +54,7 @@ public interface RpcBroker {
      * is received
      *
      * @param endpoint The endpoint that received this message
-     * @param message the concrete {@RpcMessage}
+     * @param message the concrete {@link RpcMessage}
      */
     public void publish(JsonRpcEndpoint endpoint, RpcMessage message);
 
index 9babda2bac32566dd02bec9296ad558391ed0d7a..a81c9f5fec84d9733bca894abe0a7b55a95074da 100644 (file)
@@ -18,7 +18,7 @@ import org.opendaylight.groupbasedpolicy.renderer.opflex.mit.PolicyPropertyInfo.
 /**
  * Class that provides Managed Object metadata. The {@link PolicyClassInfo} object contains metadata
  * about
- * the Managed Object, while the {@link PropertyInfo} list
+ * the Managed Object, while the {@link PolicyPropertyInfo} list
  * contained by this object provides the metadata about the
  * Managed Object properties.
  * This data must be kept consistent with the agents.