Fix Java 8 javadoc compatibility 90/17290/3
authorRobert Varga <rovarga@cisco.com>
Sat, 28 Mar 2015 17:15:06 +0000 (18:15 +0100)
committerMichal Polkorab <michal.polkorab@pantheon.sk>
Tue, 31 Mar 2015 08:13:47 +0000 (08:13 +0000)
Java 8 is more picky about javadocs -- make sure we can pass
compilation.

Change-Id: I816b25cc439cc7eadd8e073205611cb15ab95e9b
Signed-off-by: Robert Varga <rovarga@cisco.com>
19 files changed:
openflow-protocol-api/src/main/java/org/opendaylight/openflowjava/protocol/api/extensibility/DeserializerExtensionProvider.java
openflow-protocol-api/src/main/java/org/opendaylight/openflowjava/protocol/api/extensibility/DeserializerRegistry.java
openflow-protocol-api/src/main/java/org/opendaylight/openflowjava/protocol/api/extensibility/SerializerExtensionProvider.java
openflow-protocol-api/src/main/java/org/opendaylight/openflowjava/protocol/api/extensibility/SerializerRegistry.java
openflow-protocol-api/src/main/java/org/opendaylight/openflowjava/protocol/api/keys/InstructionSerializerKey.java
openflow-protocol-api/src/main/java/org/opendaylight/openflowjava/protocol/api/keys/MatchEntrySerializerKey.java
openflow-protocol-impl/src/main/java/org/opendaylight/openflowjava/protocol/impl/core/SwitchConnectionProviderImpl.java
openflow-protocol-impl/src/main/java/org/opendaylight/openflowjava/protocol/impl/serialization/SerializerRegistryImpl.java
openflow-protocol-impl/src/main/java/org/opendaylight/openflowjava/protocol/impl/util/AbstractCodeKeyMaker.java
openflow-protocol-impl/src/main/java/org/opendaylight/openflowjava/protocol/impl/util/ActionConstants.java
openflow-protocol-impl/src/main/java/org/opendaylight/openflowjava/protocol/impl/util/ActionDeserializerRegistryHelper.java
openflow-protocol-impl/src/main/java/org/opendaylight/openflowjava/protocol/impl/util/ActionSerializerRegistryHelper.java
openflow-protocol-impl/src/main/java/org/opendaylight/openflowjava/protocol/impl/util/InstructionDeserializerRegistryHelper.java
openflow-protocol-impl/src/main/java/org/opendaylight/openflowjava/protocol/impl/util/ListSerializer.java
openflow-protocol-impl/src/main/java/org/opendaylight/openflowjava/protocol/impl/util/TypeToClassInitHelper.java
openflow-protocol-impl/src/main/java/org/opendaylight/openflowjava/statistics/StatisticsCounters.java
simple-client/src/main/java/org/opendaylight/openflowjava/protocol/impl/clients/ListeningSimpleClient.java
simple-client/src/main/java/org/opendaylight/openflowjava/protocol/impl/clients/ScenarioFactory.java
util/src/main/java/org/opendaylight/openflowjava/util/ByteBufUtils.java

index abd85ba56489a20ea4d67574926816a9a948d6ff..b38b5cfacdbc46025cde504d678af2ce22b5ff41 100644 (file)
@@ -24,7 +24,7 @@ import org.opendaylight.yang.gen.v1.urn.opendaylight.openflow.protocol.rev130731
  * In case of handling multiple multiple structures of same type (actions,
  * instructions, match entries, ... ) which are differentiated by
  * vendor / experimenter subtype, vendor has to switch / choose between
- * these subtypes. <br />
+ * these subtypes.<br>
  *
  * This has to be done in this way because of experimenter headers, which
  * provide only vendor / experimenter ID. Subtype position may be different
@@ -113,4 +113,4 @@ public interface DeserializerExtensionProvider {
      */
     void registerQueuePropertyDeserializer(ExperimenterIdDeserializerKey key,
             OFDeserializer<QueueProperty> deserializer);
-}
\ No newline at end of file
+}
index 04291b8fa4e51b7181a3ccf8edc488d1d148b9e6..6970a1dc67280cf9e264065715b34ae89d8b864f 100644 (file)
@@ -45,7 +45,6 @@ public interface DeserializerRegistry {
     /**
      * Unregisters deserializer
      * @param key used for deserializer lookup
-     * @param deserializer deserializer instance
      * @return true if deserializer was removed,
      *  false if no deserializer was found under specified key
      */
index 9e73e951bf8a0065cdcc4145ffab585a5cb1807e..7d83cd8282921c54d3a05fee6b50044a8f22f4fb 100644 (file)
@@ -26,7 +26,7 @@ import org.opendaylight.yang.gen.v1.urn.opendaylight.openflow.protocol.rev130731
  * In case of handling multiple structures of same type (actions,
  * instructions, match entries, ... ) which are differentiated by
  * vendor / experimenter subtype, vendor has to switch / choose between
- * these subtypes. <br />
+ * these subtypes.<br>
  *
  * This has to be done in this way because of unknown augmentations
  * - that's why vendor has to handle it in his own implementations.
@@ -97,4 +97,4 @@ public interface SerializerExtensionProvider {
      */
     void registerMeterBandSerializer(ExperimenterIdSerializerKey<MeterBandExperimenterCase> key,
             OFSerializer<MeterBandExperimenterCase> serializer);
-}
\ No newline at end of file
+}
index 716281c40a43c22f68eb1051343179055053e698..7fb39bbd9eeb3755f4510e51959a06291e3f67cf 100644 (file)
@@ -9,22 +9,18 @@ package org.opendaylight.openflowjava.protocol.api.extensibility;
 
 import org.opendaylight.openflowjava.protocol.api.keys.MessageTypeKey;
 
-
-
-
 /**
- * Stores and handles serializers <br/>
- * K - {@link MessageTypeKey} parameter type,<br/>
+ * Stores and handles serializers<br>
+ * K - {@link MessageTypeKey} parameter type,<br>
  * S - returned serializer type
  * @author michal.polkorab
- *
  */
 public interface SerializerRegistry {
 
     /**
      * Serializer registry provisioning
      */
-    public void init();
+    void init();
 
     /**
      * @param msgTypeKey lookup key
@@ -49,7 +45,6 @@ public interface SerializerRegistry {
     /**
      * Unregisters serializer
      * @param key used for serializer lookup
-     * @param serializer serializer implementation
      * @return true if serializer was removed,
      *  false if no serializer was found under specified key
      */
index 89f5cbfead224753c5ea94f54ef9ca77ba8f750f..213b22dbf19273ed3c0362bf8ffd588cc234ae60 100644 (file)
@@ -23,7 +23,6 @@ public class InstructionSerializerKey<T extends InstructionChoice>
 
     /**
      * @param msgVersion protocol wire version
-     * @param objectType class of serialized object (Instruction.class)
      * @param instructionType type of instruction
      * @param experimenterId experimenter / vendor ID
      */
@@ -77,4 +76,4 @@ public class InstructionSerializerKey<T extends InstructionChoice>
         return super.toString() + " instructionType type: " + instructionType.getName()
                 + " vendorID: " + experimenterId;
     }
-}
\ No newline at end of file
+}
index a0867ceb1d3c055d40afc7b31c80c0538043ae03..75fcd3e60045b4b2c901155d6501e8b886985c40 100644 (file)
@@ -26,7 +26,6 @@ public final class MatchEntrySerializerKey<C extends OxmClassBase, F extends Mat
 
     /**
      * @param msgVersion protocol wire version
-     * @param objectType class of serialized object
      * @param oxmClass oxm_class (see specification)
      * @param oxmField oxm_field (see specification)
      */
@@ -95,4 +94,4 @@ public final class MatchEntrySerializerKey<C extends OxmClassBase, F extends Mat
         return super.toString() + " oxm_class: " + oxmClass.getName() + " oxm_field: "
                 + oxmField.getName() + " experimenterID: " + experimenterId;
     }
-}
\ No newline at end of file
+}
index 51b802e3d500737cf819db172f6796776ecaadd0..c2b4890dfab34f0acaa6c41ef8b6844b97f4a9c8 100644 (file)
@@ -54,8 +54,8 @@ import com.google.common.util.concurrent.ListenableFuture;
 import com.google.common.util.concurrent.SettableFuture;
 
 /**
- * Exposed class for server handling <br/>
- * C - {@link MatchEntrySerializerKey} parameter representing oxm_class (see specification) <br/>
+ * Exposed class for server handling<br>
+ * C - {@link MatchEntrySerializerKey} parameter representing oxm_class (see specification)<br>
  * F - {@link MatchEntrySerializerKey} parameter representing oxm_field (see specification)
  * @author mirehak
  * @author michal.polkorab
index 26cdda1a0b3e80f4894230326c9b7d3e4d014110..9a4c20859c92589382202afcf9f02da4064a1dbe 100644 (file)
@@ -24,8 +24,8 @@ import org.slf4j.Logger;
 import org.slf4j.LoggerFactory;
 
 /**
- * Stores and handles serializers <br/>
- * K - {@link MessageTypeKey} type <br/>
+ * Stores and handles serializers<br>
+ * K - {@link MessageTypeKey} type<br>
  * S - returned serializer type
  * @author michal.polkorab
  * @author timotej.kubas
index 45f521e1756c90342000b23228c531b6e990053e..7aff0aa7be04af5a705d00f158e8139b7188b5dd 100644 (file)
@@ -5,13 +5,10 @@
  * terms of the Eclipse Public License v1.0 which accompanies this distribution,
  * and is available at http://www.eclipse.org/legal/epl-v10.html
  */
-
 package org.opendaylight.openflowjava.protocol.impl.util;
 
-
 /**
  * @author michal.polkorab
- * @param <T> type the key maker is based on
  */
 public abstract class AbstractCodeKeyMaker implements CodeKeyMaker {
 
index d05b8cd8055602153ff19469de0705baed68c083..41791780813bca9bb29bf0efa9ee3a504775d676 100644 (file)
@@ -14,7 +14,7 @@ package org.opendaylight.openflowjava.protocol.impl.util;
  */
 public final class ActionConstants {
 
-    /** Openflow v1.0 & v1.3 OFPAT_OUTPUT code */
+    /** Openflow v1.0 and v1.3 OFPAT_OUTPUT code */
     public static final byte OUTPUT_CODE = 0;
     /** Openflow v1.0 OFPAT_SET_VLAN_VID code */
     public static final byte SET_VLAN_VID_CODE = 1;
@@ -96,7 +96,7 @@ public final class ActionConstants {
     /** Most common action length */
     public static final byte GENERAL_ACTION_LENGTH = 8;
     /** Action larger than GENERAL_ACTION_LENGTH - currently
-     *  only 16 bytes long actions for both OF v1.0 & v1.3*/
+     *  only 16 bytes long actions for both OF v1.0 and v1.3*/
     public static final byte LARGER_ACTION_LENGTH = 16;
     /** Action header size */
     public static final byte ACTION_IDS_LENGTH = 4;
index 2f48763a2ce8bb226d28054ec1126e108eb001ed..49c1b6fc14b70e08c54e0643cb900c8ecb9d2851 100644 (file)
@@ -32,8 +32,6 @@ public class ActionDeserializerRegistryHelper {
     /**
      * @param code code / value to distinguish between deserializers
      * @param deserializer deserializer instance
-     * @param deserializedObjectClass class of object that will be deserialized
-     *  by given deserializer
      */
     public void registerDeserializer(int code, OFGeneralDeserializer deserializer) {
         registry.registerDeserializer(new ActionDeserializerKey(version, code,
index 32b6ac0e9085bbc7f5cfaa609cbb652ec098be84..b622dd17edacf005cda3059931874a82d58f1349 100644 (file)
@@ -22,7 +22,6 @@ public class ActionSerializerRegistryHelper {
 
     /**
      * @param version Openflow wire version
-     * @param objectType
      * @param serializerRegistry
      */
     public ActionSerializerRegistryHelper(short version, SerializerRegistry serializerRegistry) {
@@ -40,4 +39,4 @@ public class ActionSerializerRegistryHelper {
         serializerRegistry.registerSerializer(new ActionSerializerKey<>(version,
                 actionType, null), serializer);
     }
-}
\ No newline at end of file
+}
index 44a6a884fe741ce06d2eeaa4c140acc284334827..19149e109c94973281b5598feaac9eed42743216 100644 (file)
@@ -32,8 +32,6 @@ public class InstructionDeserializerRegistryHelper {
     /**
      * @param code code / value to distinguish between deserializers
      * @param deserializer deserializer instance
-     * @param deserializedObjectClass class of object that will be deserialized
-     *  by given deserializer
      */
     public void registerDeserializer(int code, OFGeneralDeserializer deserializer) {
         registry.registerDeserializer(new InstructionDeserializerKey(version, code,
index 7925281bc9c8856827ff0baf5ed1d43909d0b6c1..7392be7612993287bcc1e65832c195bfa77e538f 100644 (file)
@@ -20,7 +20,6 @@ import org.opendaylight.yangtools.yang.binding.DataObject;
 /**
  * Serializes list items and their headers
  * @author michal.polkorab
- * @param <E>
  */
 public abstract class ListSerializer {
 
index 672cdea6afc37d60cae0810a27e90759e881711c..339edd189ccd16fae7c28ba4c529ee9cf522d0ab 100644 (file)
@@ -32,7 +32,7 @@ public class TypeToClassInitHelper {
     /**
      * Registers Class int the type to class mapping
      * @param type code value for message type / class
-     * @param clazz corresponding code -> class
+     * @param clazz class corresponding to the code
      */
     public void registerTypeToClass(short type, Class<?> clazz) {
         messageClassMap.put(new TypeToClassKey(version, type), clazz);
index 89eeff92a6ae4f765a3de26587f112358e2059bd..f3566a09385f076e0bbeb3c66c3eb1d9086fbe80 100644 (file)
@@ -186,8 +186,8 @@ public final class StatisticsCounters implements StatisticsHandler {
     /**
      * Get counter by CounterEventType
      * @param counterEventKey key to identify counter (can not be null)
-     * @return Counter object or null if counter has not been enabled
-     * @exception - IllegalArgumentException if counterEventKey is null
+     * @return Counter object or null if counter has not been enabled
+     * @throws IllegalArgumentException if counterEventKey is null
      */
     public Counter getCounter(CounterEventTypes counterEventKey) {
         if (counterEventKey == null) {
@@ -245,4 +245,4 @@ public final class StatisticsCounters implements StatisticsHandler {
             }
         }
     }
-}
\ No newline at end of file
+}
index 0288745fdc7cfd423b7177a9dc74eb635f152629..559c2279f81e7a9321eda696cc49bfb58a058edd 100644 (file)
@@ -1,3 +1,10 @@
+/*
+ * Copyright (c) 2015 Pantheon Technologies s.r.o. and others. All rights reserved.
+ *
+ * This program and the accompanying materials are made available under the
+ * terms of the Eclipse Public License v1.0 which accompanies this distribution,
+ * and is available at http://www.eclipse.org/legal/epl-v10.html
+ */
 package org.opendaylight.openflowjava.protocol.impl.clients;
 
 import io.netty.bootstrap.ServerBootstrap;
@@ -33,7 +40,6 @@ public class ListeningSimpleClient implements OFClient {
     /**
      * Constructor of the class
      *
-     * @param host address of host
      * @param port host listening port
      */
     public ListeningSimpleClient(int port) {
@@ -121,4 +127,4 @@ public class ListeningSimpleClient implements OFClient {
     public int getPort() {
         return this.port;
     }
-}
\ No newline at end of file
+}
index f0e5b9cffbf958f7de182bcaf137beee038bbc39..5b4e669aa6d8867043fc35ce6f53d139f9f09790 100644 (file)
@@ -25,8 +25,8 @@ public final class ScenarioFactory {
     }
 
     /**
-     * Creates stack with handshake needed messages.
-     * <ol> XID of messages:
+     * Creates stack with handshake needed messages. XID of messages:
+     * <ol>
      *   <li> hello sent - 00000001
      *   <li> hello waiting - 00000002
      *   <li> featuresrequest waiting - 00000003
@@ -45,8 +45,8 @@ public final class ScenarioFactory {
     }
 
     /**
-     * Creates stack with handshake needed messages.
-     * <ol> XID of messages:
+     * Creates stack with handshake needed messages. XID of messages:
+     * <ol>
      *   <li> hello sent - 00000001
      *   <li> hello waiting - 00000002
      *   <li> featuresrequest waiting - 00000003
index 43ced12463ce9be15d190035ce50949c1b9f268f..613442917afddcd0ebb9f1e34269b1b84d148407 100644 (file)
@@ -224,10 +224,11 @@ public abstract class ByteBufUtils {
     }
 
     /**
-     * Converts macAddress to byte array
+     * Converts macAddress to byte array.
+     * See also {@link org.opendaylight.yang.gen.v1.urn.ietf.params.xml.ns.yang.ietf.yang.types.rev100924.MacAddress}.
+     *
      * @param macAddress
      * @return byte representation of mac address
-     * @see {@link org.opendaylight.yang.gen.v1.urn.ietf.params.xml.ns.yang.ietf.yang.types.rev100924.MacAddress}
      */
     public static byte[] macAddressToBytes(final String macAddress) {
         final byte[] result = new byte[EncodeConstants.MAC_ADDRESS_LENGTH];
@@ -279,10 +280,11 @@ public abstract class ByteBufUtils {
     }
 
     /**
-     * Converts a MAC address represented in bytes to String
+     * Converts a MAC address represented in bytes to String.
+     * See also {@link org.opendaylight.yang.gen.v1.urn.ietf.params.xml.ns.yang.ietf.yang.types.rev100924.MacAddress}.
+     *
      * @param address
      * @return String representation of a MAC address
-     * @see {@link MacAddress}
      */
     public static String macAddressToString(final byte[] address) {
         Preconditions.checkArgument(address.length == EncodeConstants.MAC_ADDRESS_LENGTH);