commit cbd1b39f584019da9ce26a501eb91cf075ab2902
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

Sonar fixes

Rule: Use isEmpty() to check whether the collection is empty or not.

Change-Id: Ib5ab56b2263623b297db56c5cac2c2ee80e12725

providers/pcep/cli/src/main/java/org/onosproject/pcep/cli/PcepSessionCommand.java

diff --git a/providers/pcep/cli/src/main/java/org/onosproject/pcep/cli/PcepSessionCommand.java b/providers/pcep/cli/src/main/java/org/onosproject/pcep/cli/PcepSessionCommand.java
index 420dd0c..f7407ec 100644
--- a/providers/pcep/cli/src/main/java/org/onosproject/pcep/cli/PcepSessionCommand.java
+++ b/providers/pcep/cli/src/main/java/org/onosproject/pcep/cli/PcepSessionCommand.java
@@ -93,7 +93,7 @@
             pcepSessionKeySet = pcepSessionMap.keySet();
             pcepSessionIdKeySet = pcepSessionIdMap.keySet();
             if (peer != null) {
-                if (pcepSessionKeySet.size() > 0) {
+                if (!pcepSessionKeySet.isEmpty()) {
                     if (pcepSessionKeySet.contains(peer)) {
                         for (String pcepSessionPeer : pcepSessionKeySet) {
                             if (pcepSessionPeer.equals(peer)) {
@@ -116,7 +116,7 @@
                     }
                 }
             } else {
-                if (pcepSessionKeySet.size() > 0) {
+                if (!pcepSessionKeySet.isEmpty()) {
                     for (String pcepSessionPeer : pcepSessionKeySet) {
                         for (String pcepSessionId : pcepSessionIdKeySet) {
                             if (pcepSessionId.equals(pcepSessionPeer)) {
@@ -143,7 +143,7 @@
             this.pcepClientController = get(PcepClientController.class);
             Map<String, List<String>> pcepSessionFailureReasonMap = pcepClientController.getPcepExceptions();
             pcepSessionFailurekeySet = pcepSessionFailureReasonMap.keySet();
-            if (pcepSessionFailurekeySet.size() > 0) {
+            if (!pcepSessionFailurekeySet.isEmpty()) {
                 if (peer != null) {
                     if (pcepSessionFailurekeySet.contains(peer)) {
                         for (String pcepSessionPeerId : pcepSessionFailurekeySet) {
@@ -158,7 +158,7 @@
 
                 } else {
                     pcepSessionFailurekeySet = pcepSessionFailureReasonMap.keySet();
-                    if (pcepSessionFailurekeySet.size() > 0) {
+                    if (!pcepSessionFailurekeySet.isEmpty()) {
                         for (String pcepSessionPeerId : pcepSessionFailurekeySet) {
                             pcepSessionExceptions = pcepSessionFailureReasonMap.get(pcepSessionPeerId);
                             print("PeerId = %s, FailureReason = %s", pcepSessionPeerId, pcepSessionExceptions);
@@ -181,7 +181,7 @@
             this.pcepClientController = get(PcepClientController.class);
             Map<Integer, Integer> pcepErrorMsgMap = pcepClientController.getPcepErrorMsg();
             pcepErrorMsgKey = pcepErrorMsgMap.keySet();
-            if (pcepErrorMsgKey.size() > 0) {
+            if (!pcepErrorMsgKey.isEmpty()) {
                 for (Integer errorType : pcepErrorMsgKey) {
                     pcepErrorValue = pcepErrorMsgMap.get(errorType);
                     pcepErrorType = PcepErrorType.values()[errorType];