Enable checkstyle Javadoc Comment Style rule
Turn on the Javadoc Comment Style rule and fix
any violations that come up. This is related to
requiring javadoc description comments to end
with a '.'
Change-Id: I19cf1b9f00d3040987e0b4c8b39461e2e0ad66b5
diff --git a/src/main/java/net/onrc/onos/core/linkdiscovery/internal/LinkDiscoveryManager.java b/src/main/java/net/onrc/onos/core/linkdiscovery/internal/LinkDiscoveryManager.java
index 4cb8bc7..9b913c1 100644
--- a/src/main/java/net/onrc/onos/core/linkdiscovery/internal/LinkDiscoveryManager.java
+++ b/src/main/java/net/onrc/onos/core/linkdiscovery/internal/LinkDiscoveryManager.java
@@ -101,14 +101,14 @@
* <p/>
* Invariants:
* -portLinks and switchLinks will not contain empty Sets outside of
- * critical sections
+ * critical sections.
* -portLinks contains LinkTuples where one of the src or dst
- * SwitchPortTuple matches the map key
+ * SwitchPortTuple matches the map key.
* -switchLinks contains LinkTuples where one of the src or dst
- * SwitchPortTuple's id matches the switch id
+ * SwitchPortTuple's id matches the switch id.
* -Each LinkTuple will be indexed into switchLinks for both
- * src.id and dst.id, and portLinks for each src and dst
- * -The updates queue is only added to from within a held write lock
+ * src.id and dst.id, and portLinks for each src and dst.
+ * -The updates queue is only added to from within a held write lock.
*/
@LogMessageCategory("Network Topology")
public class LinkDiscoveryManager
@@ -181,17 +181,17 @@
protected Map<Long, IOnosRemoteSwitch> remoteSwitches;
/**
- * Map from link to the most recent time it was verified functioning
+ * Map from link to the most recent time it was verified functioning.
*/
protected Map<Link, LinkInfo> links;
/**
- * Map from switch id to a set of all links with it as an endpoint
+ * Map from switch id to a set of all links with it as an endpoint.
*/
protected Map<Long, Set<Link>> switchLinks;
/**
- * Map from a id:port to the set of links containing it as an endpoint
+ * Map from a id:port to the set of links containing it as an endpoint.
*/
protected Map<NodePortTuple, Set<Link>> portLinks;
@@ -203,8 +203,10 @@
protected volatile boolean shuttingDown = false;
- /* topology aware components are called in the order they were added to the
- * the array */
+ /**
+ * Topology aware components are called in the order they were added to the
+ * the array.
+ */
protected ArrayList<ILinkDiscoveryListener> linkDiscoveryAware;
protected class LinkUpdate extends LDUpdate {
@@ -252,7 +254,7 @@
protected LinkedBlockingQueue<NodePortTuple> quarantineQueue;
protected LinkedBlockingQueue<NodePortTuple> maintenanceQueue;
/**
- * Quarantine task
+ * Quarantine task.
*/
protected SingletonTask bddpTask;
protected static final int BDDP_TASK_INTERVAL = 100; // 100 ms.
@@ -274,7 +276,7 @@
}
/**
- * Get the LLDP timeout value in seconds
+ * Get the LLDP timeout value in seconds.
*
* @return LLDP timeout value in seconds
*/
@@ -498,7 +500,7 @@
}
/**
- * Send LLDP on known ports
+ * Send LLDP on known ports.
*/
protected void discoverOnKnownLinkPorts() {
// Copy the port set.
@@ -520,7 +522,7 @@
}
/**
- * Learn remote switches when running as a distributed controller ONOS
+ * Learn remote switches when running as a distributed controller ONOS.
*/
protected IOFSwitch addRemoteSwitch(long sw, short port) {
IOnosRemoteSwitch remotesw = null;
@@ -697,7 +699,7 @@
}
/**
- * Send LLDPs to all switch-ports
+ * Send LLDPs to all switch-ports.
*/
protected void discoverOnAllPorts() {
if (log.isTraceEnabled()) {
@@ -1409,7 +1411,7 @@
}
/**
- * We send out LLDP messages when a switch is added to discover the topology
+ * We send out LLDP messages when a switch is added to discover the topology.
*
* @param sw The IOFSwitch that connected to the controller
*/
@@ -1631,7 +1633,7 @@
}
/**
- * Register a link discovery aware component
+ * Register a link discovery aware component.
*
* @param linkDiscoveryAwareComponent
*/
@@ -1641,7 +1643,7 @@
}
/**
- * Deregister a link discovery aware component
+ * Deregister a link discovery aware component.
*
* @param linkDiscoveryAwareComponent
*/