Remove unneeded boxing 68/92368/3
authorRobert Varga <robert.varga@pantheon.tech>
Wed, 2 Sep 2020 08:58:31 +0000 (10:58 +0200)
committerRobert Varga <robert.varga@pantheon.tech>
Mon, 7 Sep 2020 08:12:52 +0000 (10:12 +0200)
Use of Integer creates a box-and-unbox operation when coupled
with parseInt() and switch(), just use a plain int.

Change-Id: I62b9d925edf44a22422d14b00b0f413e83ccaf6b
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>
test-provider/src/main/java/org/opendaylight/openflowplugin/test/OpenflowPluginBulkGroupTransactionProvider.java
test-provider/src/main/java/org/opendaylight/openflowplugin/test/OpenflowpluginMeterTestCommandProvider.java

index 21bb83cb37e47ba8eca763406fdd5736bba6effe..255771a822ced67f3f0ee3a21094391b39e8e7b6 100644 (file)
@@ -565,7 +565,7 @@ public class OpenflowPluginBulkGroupTransactionProvider implements CommandProvid
             ci.println("User node added" + nref);
             createUserNode(nref);
         }
-        Integer count = Integer.parseInt(ci.nextArgument());
+        int count = Integer.parseInt(ci.nextArgument());
         switch (count) {
             case 1:
                 GroupBuilder group = createTestGroup("a7", "g1", "add", "1");
@@ -610,7 +610,7 @@ public class OpenflowPluginBulkGroupTransactionProvider implements CommandProvid
             ci.println("User node added" + nref);
             createUserNode(nref);
         }
-        Integer count = Integer.parseInt(ci.nextArgument());
+        int count = Integer.parseInt(ci.nextArgument());
         switch (count) {
             case 1:
                 GroupBuilder group = createTestGroup("a4", "g1", "modify", "1");
@@ -653,7 +653,7 @@ public class OpenflowPluginBulkGroupTransactionProvider implements CommandProvid
             createUserNode(nref);
         }
 
-        Integer count = Integer.parseInt(ci.nextArgument());
+        int count = Integer.parseInt(ci.nextArgument());
         switch (count) {
             case 1:
                 GroupBuilder group = createTestGroup("a2", "g1", "remove", "1");
index c0be3303b98d281a5788d61a88f52537f683ca2b..74be5f753016e00b5fefa9b8d617baed664ac17d 100644 (file)
@@ -251,7 +251,7 @@ public class OpenflowpluginMeterTestCommandProvider implements CommandProvider {
             createUserNode(nref);
         }
 
-        Integer count = Integer.parseInt(ci.nextArgument());
+        int count = Integer.parseInt(ci.nextArgument());
 
         ReadWriteTransaction modification = dataBroker.newReadWriteTransaction();
         switch (count) {
@@ -345,7 +345,7 @@ public class OpenflowpluginMeterTestCommandProvider implements CommandProvider {
             ci.println("User node added" + nref);
             createUserNode(nref);
         }
-        Integer count = Integer.parseInt(ci.nextArgument());
+        int count = Integer.parseInt(ci.nextArgument());
         switch (count) {
             case 1:
                 createTestMeters("1", "add");
@@ -448,7 +448,7 @@ public class OpenflowpluginMeterTestCommandProvider implements CommandProvider {
             createUserNode(nref);
         }
 
-        Integer count = Integer.parseInt(ci.nextArgument());
+        int count = Integer.parseInt(ci.nextArgument());
         switch (count) {
             case 1:
                 createTestMeters("1", "modify");