removed code smells 51/105151/1
authorRupinder <rupinsi1@in.ibm.com>
Mon, 6 Apr 2020 13:28:23 +0000 (18:58 +0530)
committerRupinder <rupinsi1@in.ibm.com>
Mon, 6 Apr 2020 13:28:35 +0000 (18:58 +0530)
Removed some unused imports and 2 statments from one line

Issue-ID: PORTAL-869
Change-Id: I99f97a7a7c16a4ff2d28c40cfd88546dfe799a29
Signed-off-by: Rupindewr<rupinsi1@in.ibm.com>
music-core/src/main/java/org/onap/music/main/CorePropertiesLoader.java
music-core/src/main/java/org/onap/music/main/DeadlockDetectionUtil.java
music-rest/src/main/java/org/onap/music/rest/RestMusicQAPI.java

index c386667..2e5ca68 100644 (file)
@@ -24,8 +24,6 @@ package org.onap.music.main;
 
 import java.util.Properties;
 
-import org.onap.music.eelf.logging.EELFLoggerDelegate;
-
 public class CorePropertiesLoader {
 
     public static void loadProperties(Properties properties) {
index 4c9a74b..23ccca7 100644 (file)
@@ -39,7 +39,8 @@ public class DeadlockDetectionUtil {
                @Override
                public String toString() {
                        StringBuffer sb = new StringBuffer();
-                       for (Node link : links) sb.append(link.id);
+                       for (Node link : links)
+                               sb.append(link.id);
                        return "Node [id=" + id + ", links=" + sb.toString() + ", visited=" + visited + ", onStack=" + onStack + "]";
                }
 
@@ -109,13 +110,17 @@ public class DeadlockDetectionUtil {
        }
 
        private boolean findCycle(Node currentNode) {
-               if (currentNode==null) return false;
-               if (currentNode.isOnStack()) return true;
-               if (currentNode.isVisited()) return false;
+               if (currentNode==null)
+                       return false;
+               if (currentNode.isOnStack())
+                       return true;
+               if (currentNode.isVisited())
+                       return false;
                currentNode.setOnStack(true);
                currentNode.setVisited(true);
                for (Node childNode : currentNode.getLinks()) {
-                       if (findCycle(childNode)) return true;
+                       if (findCycle(childNode))
+                               return true;
                }
                currentNode.setOnStack(false);
                return false;
index 4def0e4..63b72cd 100755 (executable)
@@ -174,9 +174,11 @@ public class RestMusicQAPI {
                 clusteringKey=clusteringKey.replaceAll("[\\(]+","");
                 clusteringKey=clusteringKey.replaceAll("[\\)]+","");
                 clusteringKey = clusteringKey.trim();
-                if (clusteringKey.indexOf(',') == 0) clusteringKey=clusteringKey.substring(1);
+                if (clusteringKey.indexOf(',') == 0)
+                                       clusteringKey=clusteringKey.substring(1);
                 clusteringKey = clusteringKey.trim();
-                if (clusteringKey.equals(",") ) clusteringKey=""; // print error if needed    ( ... ),)
+                if (clusteringKey.equals(",") )
+                                       clusteringKey=""; // print error if needed    ( ... ),)
             }
         }