commit 7d21c0a0436a6c18d350d1b09088ea87aaa95f4f
author Pavlin Radoslavov <pavlin@onlab.us> Thu Apr 10 10:32:59 2014 -0700
committer Pavlin Radoslavov <pavlin@onlab.us> Thu Apr 10 10:32:59 2014 -0700
tree 581805e7c1cd11314f8fc4f2d2a39c41f2e03d9a
parent b29e62622f68bfe3aa4516e81ff29fe3b18e9cf8

Fix issues found by FindBugs: NM_METHOD_NAMING_CONVENTION

http://findbugs.sourceforge.net/bugDescriptions.html#NM_METHOD_NAMING_CONVENTION

Renamed methods:
AddToSuppressLLDPs -> addToSuppressLLDPs
RemoveFromSuppressLLDPs -> removeFromSuppressLLDPs

Change-Id: I2a5155433c0503c5efa067867392925ea461cbf4

src/main/java/net/onrc/onos/core/linkdiscovery/ILinkDiscoveryService.java

diff --git a/src/main/java/net/onrc/onos/core/linkdiscovery/ILinkDiscoveryService.java b/src/main/java/net/onrc/onos/core/linkdiscovery/ILinkDiscoveryService.java
index dce8be0..14e78d8 100644
--- a/src/main/java/net/onrc/onos/core/linkdiscovery/ILinkDiscoveryService.java
+++ b/src/main/java/net/onrc/onos/core/linkdiscovery/ILinkDiscoveryService.java
@@ -59,12 +59,12 @@
     /**
      * Adds a switch port to suppress lldp set
      */
-    public void AddToSuppressLLDPs(long sw, short port);
+    public void addToSuppressLLDPs(long sw, short port);
 
     /**
      * Removes a switch port from suppress lldp set
      */
-    public void RemoveFromSuppressLLDPs(long sw, short port);
+    public void removeFromSuppressLLDPs(long sw, short port);
 
     /**
      * Get the set of quarantined ports on a switch

src/main/java/net/onrc/onos/core/linkdiscovery/internal/LinkDiscoveryManager.java

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 2267d06..4cb8bc7 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
@@ -296,7 +296,7 @@
      * Remove any known links on the switch port.
      */
     @Override
-    public void AddToSuppressLLDPs(long sw, short port) {
+    public void addToSuppressLLDPs(long sw, short port) {
         NodePortTuple npt = new NodePortTuple(sw, port);
         this.suppressLinkDiscovery.add(npt);
         deleteLinksOnPort(npt, "LLDP suppressed.");
@@ -307,7 +307,7 @@
      * Discover links on that switchport.
      */
     @Override
-    public void RemoveFromSuppressLLDPs(long sw, short port) {
+    public void removeFromSuppressLLDPs(long sw, short port) {
         NodePortTuple npt = new NodePortTuple(sw, port);
         this.suppressLinkDiscovery.remove(npt);
         discover(npt);

src/main/java/net/onrc/onos/core/topology/NetworkGraphPublisher.java

diff --git a/src/main/java/net/onrc/onos/core/topology/NetworkGraphPublisher.java b/src/main/java/net/onrc/onos/core/topology/NetworkGraphPublisher.java
index bc72e00..e00e3c9 100644
--- a/src/main/java/net/onrc/onos/core/topology/NetworkGraphPublisher.java
+++ b/src/main/java/net/onrc/onos/core/topology/NetworkGraphPublisher.java
@@ -157,7 +157,7 @@
     public void switchPortAdded(Long switchId, OFPhysicalPort port) {
         PortEvent portEvent = new PortEvent(switchId, (long) port.getPortNumber());
         networkGraphDiscoveryInterface.putPortDiscoveryEvent(portEvent);
-        linkDiscovery.RemoveFromSuppressLLDPs(switchId, port.getPortNumber());
+        linkDiscovery.removeFromSuppressLLDPs(switchId, port.getPortNumber());
     }
 
     @Override
@@ -185,7 +185,7 @@
         for (OFPhysicalPort port : sw.getPorts()) {
             // Allow links to be discovered on this port now that it's
             // in the database
-            linkDiscovery.RemoveFromSuppressLLDPs(sw.getId(), port.getPortNumber());
+            linkDiscovery.removeFromSuppressLLDPs(sw.getId(), port.getPortNumber());
         }
     }