commit 4dda81402d3efcf2ad6472bb019350e13c509cdd
author Ray Milkey <ray@opennetworking.org> Fri Jun 08 08:55:37 2018 -0700
committer Ray Milkey <ray@opennetworking.org> Sun Jun 10 05:04:05 2018 +0000
tree db1274f69d4e4a57139fadae3678005509a57bc3
parent 08392f45ecb533f2271acc28b4414e07d71fcb47

Sonar suggestion - no need for a null pointer check since it was already done

Change-Id: I123bd47279ebf44cd3d04f73035ade914288ce9c

drivers/cisco/rest/src/main/java/org/onosproject/drivers/cisco/rest/ControllerConfigCiscoImpl.java

diff --git a/drivers/cisco/rest/src/main/java/org/onosproject/drivers/cisco/rest/ControllerConfigCiscoImpl.java b/drivers/cisco/rest/src/main/java/org/onosproject/drivers/cisco/rest/ControllerConfigCiscoImpl.java
index 7f5c37d..025c96b 100644
--- a/drivers/cisco/rest/src/main/java/org/onosproject/drivers/cisco/rest/ControllerConfigCiscoImpl.java
+++ b/drivers/cisco/rest/src/main/java/org/onosproject/drivers/cisco/rest/ControllerConfigCiscoImpl.java
@@ -105,21 +105,16 @@
             throw new NullPointerException("Response is null");
         }
 
-        if (response != null) {
-            try {
-                ObjectMapper om = new ObjectMapper();
-                JsonNode json = om.readTree(response);
-                //TODO parse error messages.
-                if (json.has("error")) {
-                    log.error("{} Response has IllegalStateException Error", deviceId);
-                    return;
-                }
-            } catch (IOException e) {
-                log.error("Exception thrown", e);
+        try {
+            ObjectMapper om = new ObjectMapper();
+            JsonNode json = om.readTree(response);
+            //TODO parse error messages.
+            if (json.has("error")) {
+                log.error("{} Response has IllegalStateException Error", deviceId);
+                return;
             }
-        } else {
-            log.error("Can't reach the deviceId {}", deviceId);
+        } catch (IOException e) {
+            log.error("Exception thrown", e);
         }
     }
-
 }