From: Rupinder Date: Mon, 6 Apr 2020 13:28:23 +0000 (+0530) Subject: removed code smells X-Git-Tag: 3.2.38~5^2 X-Git-Url: https://gerrit.onap.org/r/gitweb?p=music.git;a=commitdiff_plain;h=10d74ffbf1feb6eab011773cfe091f8610d7e736 removed code smells Removed some unused imports and 2 statments from one line Issue-ID: PORTAL-869 Change-Id: I99f97a7a7c16a4ff2d28c40cfd88546dfe799a29 Signed-off-by: Rupindewr --- diff --git a/music-core/src/main/java/org/onap/music/main/CorePropertiesLoader.java b/music-core/src/main/java/org/onap/music/main/CorePropertiesLoader.java index c3866672..2e5ca689 100644 --- a/music-core/src/main/java/org/onap/music/main/CorePropertiesLoader.java +++ b/music-core/src/main/java/org/onap/music/main/CorePropertiesLoader.java @@ -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) { diff --git a/music-core/src/main/java/org/onap/music/main/DeadlockDetectionUtil.java b/music-core/src/main/java/org/onap/music/main/DeadlockDetectionUtil.java index 4c9a74b7..23ccca7f 100644 --- a/music-core/src/main/java/org/onap/music/main/DeadlockDetectionUtil.java +++ b/music-core/src/main/java/org/onap/music/main/DeadlockDetectionUtil.java @@ -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; diff --git a/music-rest/src/main/java/org/onap/music/rest/RestMusicQAPI.java b/music-rest/src/main/java/org/onap/music/rest/RestMusicQAPI.java index 4def0e45..63b72cda 100755 --- a/music-rest/src/main/java/org/onap/music/rest/RestMusicQAPI.java +++ b/music-rest/src/main/java/org/onap/music/rest/RestMusicQAPI.java @@ -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 ( ... ),) } }