commit | cbd1b39f584019da9ce26a501eb91cf075ab2902 | [log] [tgz] |
---|---|---|
author | Jon Hall <jhall@ciena.com> | Wed Jan 18 20:15:44 2017 -0800 |
committer | Ray Milkey <ray@onlab.us> | Thu Jan 19 20:24:18 2017 +0000 |
tree | 59a29576180e3242864c1f157e74c82f32487396 | |
parent | 1745e5abbd4cf908579d79af77c3095970095d10 [diff] |
Sonar fixes Rule: Use isEmpty() to check whether the collection is empty or not. Change-Id: Ib5ab56b2263623b297db56c5cac2c2ee80e12725
diff --git a/protocols/isis/ctl/src/main/java/org/onosproject/isis/controller/impl/Controller.java b/protocols/isis/ctl/src/main/java/org/onosproject/isis/controller/impl/Controller.java index 406d09c..ae9504b 100644 --- a/protocols/isis/ctl/src/main/java/org/onosproject/isis/controller/impl/Controller.java +++ b/protocols/isis/ctl/src/main/java/org/onosproject/isis/controller/impl/Controller.java
@@ -127,7 +127,7 @@ configPacket[1] = numberOfInterface; //First time configuration if (processes == null) { - if (isisProcesses.size() > 0) { + if (!isisProcesses.isEmpty()) { processes = isisProcesses; connectPeer(); } @@ -337,7 +337,7 @@ } interfaceList.add(isisInterface); } - if (interfaceList.size() > 0) { + if (!interfaceList.isEmpty()) { IsisProcess process = new DefaultIsisProcess(); process.setProcessId(jsonNode.path(IsisConstants.PROCESSESID).asText()); process.setIsisInterfaceList(interfaceList);
diff --git a/protocols/isis/ctl/src/main/java/org/onosproject/isis/controller/impl/IsisMessageDecoder.java b/protocols/isis/ctl/src/main/java/org/onosproject/isis/controller/impl/IsisMessageDecoder.java index b75bc3f..43d95d8 100644 --- a/protocols/isis/ctl/src/main/java/org/onosproject/isis/controller/impl/IsisMessageDecoder.java +++ b/protocols/isis/ctl/src/main/java/org/onosproject/isis/controller/impl/IsisMessageDecoder.java
@@ -73,7 +73,7 @@ isisMessageList.add(message); } } - return (isisMessageList.size() > 0) ? isisMessageList : null; + return (!isisMessageList.isEmpty()) ? isisMessageList : null; } /**