Removed usused VppPollOperDataImpl.updateOperational() return value
authorMarek Gradzki <[email protected]>
Fri, 4 Mar 2016 09:42:24 +0000 (10:42 +0100)
committerMarek Gradzki <[email protected]>
Fri, 18 Mar 2016 09:05:27 +0000 (10:05 +0100)
Change-Id: I6eaefd0448367abdc5774d931cb22caf73166662
Signed-off-by: Marek Gradzki <[email protected]>
v3po/impl/src/main/java/io/fd/honeycomb/v3po/impl/V3poApiRequest.java
v3po/impl/src/main/java/io/fd/honeycomb/v3po/impl/VppPollOperDataImpl.java

index 5de4cfd..5416b37 100644 (file)
@@ -76,7 +76,6 @@ import org.slf4j.LoggerFactory;
  */
 public class V3poApiRequest {
     private static final Logger LOG = LoggerFactory.getLogger(V3poApiRequest.class);
-    public String ifNames = "";
     private final VppPollOperDataImpl caller;
     private final vppApi api;
 
@@ -451,7 +450,5 @@ public class V3poApiRequest {
                                              swIf.vtrOp, swIf.vtrPushDot1q,
                                              swIf.vtrTag1, swIf.vtrTag2, stats);
         writeToIfState(iid, intf);
-
-        ifNames += " " + swIf.interfaceName;
     }
 }
index e5cb13f..aa607c7 100644 (file)
@@ -66,10 +66,9 @@ public class VppPollOperDataImpl implements V3poService {
 
     /**
      * TODO-ADD-JAVADOC.
-     * Update operational data and return string of space separated
-     * interfaces names
+     * Updates operational data
      */
-    public String updateOperational() {
+    public void updateOperational() {
         V3poApiRequest req = new V3poApiRequest(api, this);
         version = api.getVppVersion();
 
@@ -118,17 +117,14 @@ public class VppPollOperDataImpl implements V3poService {
 
         // write to oper
         writeVppState(VPP_STATE, stateBuilder.build());
-
-        return req.ifNames;
     }
 
     @Override
     public Future<RpcResult<VppPollOperDataOutput>> vppPollOperData() {
-        String ifNames = updateOperational();
-
+        updateOperational();
 
         VppPollOperDataOutput output = new VppPollOperDataOutputBuilder()
-            .setStatus(new Long(1)).build();
+            .setStatus(Long.valueOf(1)).build();
 
         return RpcResultBuilder.success(output).buildFuture();
     }