Merge "some code smells removed"
authorBharath Balasubramanian <bharathb@research.att.com>
Wed, 8 Apr 2020 16:54:26 +0000 (16:54 +0000)
committerGerrit Code Review <gerrit@onap.org>
Wed, 8 Apr 2020 16:54:26 +0000 (16:54 +0000)
docs/index.rst
docs/release-notes.rst
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
music-rest/src/main/resources/application.properties [moved from music-core/src/main/resources/application.properties with 100% similarity]

index 437a50d..1982263 100755 (executable)
@@ -1,6 +1,7 @@
 .. This work is licensed under a Creative Commons Attribution 4.0 International License.
 .. http://creativecommons.org/licenses/by/4.0
 .. Copyright 2017 AT&T Intellectual Property.  All rights reserved.
+.. _master_index:
 
 Music Developer Documentation
 =============================
index 9387c20..e736d84 100644 (file)
@@ -1,5 +1,6 @@
 .. This work is licensed under a Creative Commons Attribution 4.0 International License.
 .. http://creativecommons.org/licenses/by/4.0
+.. _release_notes:
 
 
 Release Notes
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    ( ... ),)
             }
         }