]> source.dussan.org Git - sonarqube.git/commitdiff
Fix coding violations
authorSimon Brandhof <simon.brandhof@gmail.com>
Thu, 3 Nov 2011 14:34:10 +0000 (15:34 +0100)
committerSimon Brandhof <simon.brandhof@gmail.com>
Thu, 3 Nov 2011 14:34:18 +0000 (15:34 +0100)
20 files changed:
plugins/sonar-core-plugin/src/main/java/org/sonar/plugins/core/security/ApplyProjectRolesDecorator.java
plugins/sonar-core-plugin/src/main/java/org/sonar/plugins/core/sensors/WeightedViolationsDecorator.java
plugins/sonar-dbcleaner-plugin/src/main/java/org/sonar/plugins/dbcleaner/runner/DefaultPurgeContext.java
plugins/sonar-design-plugin/src/main/java/org/sonar/plugins/design/ui/page/client/DependencyInfo.java
plugins/sonar-jacoco-plugin/src/main/java/org/sonar/plugins/jacoco/JacocoAntInitializer.java
plugins/sonar-squid-java-plugin/src/main/java/org/sonar/plugins/squid/bridges/DependencyIndex.java
plugins/sonar-squid-java-plugin/src/main/java/org/sonar/plugins/squid/bridges/ResourceIndex.java
sonar-application/src/main/java/org/sonar/application/JettyEmbedder.java
sonar-duplications/src/main/java/org/sonar/duplications/token/TokenQueue.java
sonar-graph/src/main/java/org/sonar/graph/IncrementalCyclesAndFESSolver.java
sonar-plugin-api/src/main/java/org/sonar/api/profiles/RulesProfile.java
sonar-plugin-api/src/main/java/org/sonar/api/utils/DateUtils.java
sonar-server/src/main/java/org/sonar/server/charts/ChartsServlet.java
sonar-server/src/main/java/org/sonar/server/charts/deprecated/CustomBarRenderer.java
sonar-server/src/main/java/org/sonar/server/configuration/Backup.java
sonar-server/src/main/java/org/sonar/server/configuration/ProfilesBackup.java
sonar-server/src/main/java/org/sonar/server/filters/FilterResult.java
sonar-server/src/main/java/org/sonar/server/plugins/BatchResourcesServlet.java
sonar-server/src/main/java/org/sonar/server/plugins/StaticResourcesServlet.java
sonar-server/src/main/java/org/sonar/server/startup/GwtPublisher.java

index df4f66f9b048ace2892957216e9f8c6cbd83dac8..883a7d41bf79c36a8d54739f26fbfa27ba1cb9fb 100644 (file)
@@ -29,7 +29,7 @@ public class ApplyProjectRolesDecorator implements Decorator {
 
   private RoleManager roleManager;
 
-  protected ApplyProjectRolesDecorator(RoleManager roleManager) {
+  ApplyProjectRolesDecorator(RoleManager roleManager) {
     this.roleManager = roleManager;
   }
 
index a0b636d14810a659229d8cb6e3c7ce1d706bcff9..330cf8fe3651fdfb7a551db1e1b4af3bc5f7bcd8 100644 (file)
@@ -55,7 +55,7 @@ public class WeightedViolationsDecorator implements Decorator {
   /**
    * for unit tests
    */
-  protected WeightedViolationsDecorator(Map<RulePriority, Integer> weights) {
+  WeightedViolationsDecorator(Map<RulePriority, Integer> weights) {
     this.weights = weights;
   }
 
index ea49d64e485790088533a412074eebede7e25bda..adb577ed4dfb79f1dca41ef4ed987b2a4b41e6fc 100644 (file)
@@ -61,6 +61,9 @@ public final class DefaultPurgeContext implements org.sonar.api.batch.PurgeConte
     return project;
   }
 
+  /**
+   * @deprecated 
+   */
   @Deprecated
   public Integer getLastSnapshotId() {
     return currentSid;
index 220479188576f3245bed5eef7dbc95d70a787185..412322fb7797ad552b1e09f7415f5f25e47abc8f 100644 (file)
@@ -37,7 +37,7 @@ import org.sonar.wsclient.services.Resource;
 
 import java.util.List;
 
-public class DependencyInfo extends Composite {
+public final class DependencyInfo extends Composite {
 
   private static DependencyInfo INSTANCE = new DependencyInfo();
 
index e74e2a7a3c810d29252371b8e4b63e014d0fbf76..83584e2b123880193ed1cc340b3e5ed5d6ac51e1 100644 (file)
@@ -26,6 +26,7 @@ import org.sonar.api.batch.SupportedEnvironment;
 import org.sonar.api.resources.Project;
 
 import java.util.Hashtable;
+import java.util.Map;
 
 @SupportedEnvironment("ant")
 public class JacocoAntInitializer extends Initializer implements CoverageExtension {
@@ -48,7 +49,7 @@ public class JacocoAntInitializer extends Initializer implements CoverageExtensi
 
   @Override
   public void execute(org.sonar.api.resources.Project project) {
-    Hashtable<String, Target> hastable = antProject.getTargets();
+    Map<String, Target> hastable = antProject.getTargets();
     String jvmArg = configuration.getJvmArgument();
     String[] names = configuration.getAntTargets();
     for (String name : names) {
index f46c399e9b4067e87b11c610e603621cb50f2ac0..8ecf0417a0db226943a492d051c23687f8aab35f 100644 (file)
@@ -25,4 +25,5 @@ import org.sonar.graph.Edge;
 import java.util.HashMap;
 
 public final class DependencyIndex extends HashMap<Edge, Dependency> {
+  private static final long serialVersionUID = -3915853089587077911L;
 }
index ba49a80d6de7803a7ecd58799590e23bfa633317..dcb0a150e8e8a051424ebc8ffc93141d0a5fa610 100644 (file)
@@ -39,6 +39,8 @@ import java.util.HashMap;
 
 public final class ResourceIndex extends HashMap<SourceCode, Resource> {
 
+  private static final long serialVersionUID = -918346378374943773L;
+
   public ResourceIndex loadSquidResources(Squid squid, SensorContext context, Project project) {
     loadSquidProject(squid, project);
     loadSquidPackages(squid, context);
index 713ecab0267b57925de01021e0bc2db4740bb83c..309e3a40bcf5eb52b111a5b4063507946d5f7400 100644 (file)
@@ -175,7 +175,7 @@ public class JettyEmbedder {
     return null;
   }
 
-  protected Server getServer() {
+  Server getServer() {
     return server;
   }
 
index 6fcd544e27ee828f9e15abeef50d926ffa4df978..a825b83e250d42024ec1cbd30d9dca2fbcd9b656 100644 (file)
  */
 package org.sonar.duplications.token;
 
-import java.util.Iterator;
-import java.util.LinkedList;
-import java.util.List;
-import java.util.ListIterator;
+import java.util.*;
 
 public class TokenQueue implements Iterable<Token> {
 
-  private final LinkedList<Token> tokenQueue;
+  private final Deque<Token> tokenQueue;
 
   public TokenQueue(List<Token> tokenList) {
     tokenQueue = new LinkedList<Token>(tokenList);
index 051cdc92f1d1c5cf931486a557f157fe2e8c4cba..e90cb46a55b1a797673a64aa632773e0a453d44f 100644 (file)
@@ -26,7 +26,6 @@ import java.util.Set;
 public class IncrementalCyclesAndFESSolver<V> {
 
   private Set<Cycle> cycles = new HashSet<Cycle>();
-  private Set<Edge> edgesToExclude = new HashSet<Edge>();
   private long searchCyclesCalls = 0;
   private static final int DEFAULT_MAX_SEARCH_DEPTH_AT_FIRST = 3;
   private static final int DEFAULT_MAX_CYCLES_TO_FOUND_BY_ITERATION = 100;
@@ -46,7 +45,7 @@ public class IncrementalCyclesAndFESSolver<V> {
     searchCyclesCalls += cycleDetector.getSearchCyclesCalls();
     cycles.addAll(cycleDetector.getCycles());
     solver = new MinimumFeedbackEdgeSetSolver(cycles);
-    edgesToExclude = solver.getEdges();
+    Set<Edge> edgesToExclude = solver.getEdges();
 
     do {
       iterations++;
index 0455940f88f90975c4e92ebf7c4c989d3be1e6dd..20c3b4cd571f57e928d26c8880667ea90c867c17 100644 (file)
@@ -25,8 +25,6 @@ import org.apache.commons.collections.Transformer;
 import org.apache.commons.lang.StringUtils;
 import org.apache.commons.lang.builder.EqualsBuilder;
 import org.apache.commons.lang.builder.HashCodeBuilder;
-import org.apache.commons.lang.builder.ReflectionToStringBuilder;
-import org.apache.commons.lang.builder.ToStringStyle;
 import org.sonar.api.database.model.ResourceModel;
 import org.sonar.api.rules.ActiveRule;
 import org.sonar.api.rules.Rule;
@@ -143,20 +141,20 @@ public class RulesProfile implements Cloneable {
     this.name = s;
     return this;
   }
-  
+
   public int getVersion() {
     return version;
   }
-  
+
   public RulesProfile setVersion(int version) {
     this.version = version;
     return this;
   }
-  
+
   public Boolean getUsed() {
     return used;
   }
-  
+
   public RulesProfile setUsed(Boolean used) {
     this.used = used;
     return this;
@@ -308,6 +306,7 @@ public class RulesProfile implements Cloneable {
 
   /**
    * Note: disabled rules are excluded.
+   *
    * @return the list of active rules for a given severity
    */
   public List<ActiveRule> getActiveRules(RulePriority severity) {
@@ -344,6 +343,7 @@ public class RulesProfile implements Cloneable {
 
   /**
    * Note: disabled rules are excluded.
+   *
    * @return an active rule from a plugin key and a rule key if the rule is activated, null otherwise
    */
   public ActiveRule getActiveRule(String repositoryKey, String ruleKey) {
index c41d5a3684f73078c4a4f052a6a8cbcc599745b4..cd6f77b4f5fc468f6f7a47d5479fa432382d285b 100644 (file)
@@ -67,6 +67,7 @@ public final class DateUtils {
   }
 
   static class ThreadSafeDateFormat extends DateFormat {
+    private static final long serialVersionUID = -8856468429474634301L;
     private final String format;
 
     ThreadSafeDateFormat(String format) {
index c4e2fda39bb80dcf9d5bae100d41b29f43752efb..3f9ef2d756c795723a72cbb01282424162b62785 100644 (file)
@@ -42,6 +42,7 @@ import java.util.Map;
 public class ChartsServlet extends HttpServlet {
 
   private static final Logger LOG = LoggerFactory.getLogger(ChartsServlet.class);
+  private static final long serialVersionUID = 669857447198433893L;
 
   @Override
   public void doGet(HttpServletRequest request, HttpServletResponse response)
index 7bc2e1fa0dcdbde8c2af91ed15b32e1821c837f0..f149053d34ef776a08d4f489aef3987520f8e379 100644 (file)
@@ -25,6 +25,7 @@ import java.awt.*;
 
 public class CustomBarRenderer extends BarRenderer {
 
+  private static final long serialVersionUID = -1598934457150811079L;
   public static final Paint[] COLORS = new Paint[]{Color.red, Color.blue, Color.green,
       Color.yellow, Color.orange, Color.cyan,
       Color.magenta, Color.blue};
index 635f256bb794783f5e4f5c987c57f2bf1b266bf6..b1cc6b5502b35969760a24794ad2c7d646bab701 100644 (file)
@@ -61,7 +61,10 @@ public class Backup {
     backupables.add(new ProfilesBackup(session));
   }
 
-  protected Backup(List<Backupable> backupables) {
+  /**
+   * For unit tests
+   */
+  Backup(List<Backupable> backupables) {
     this();
     this.backupables = backupables;
   }
index 958fe22220b74c7da9f5e7ddbcda21b46038889a..daea84804a55320d6d95570c6514ea0bb1872f82 100644 (file)
@@ -45,7 +45,10 @@ public class ProfilesBackup implements Backupable {
     this.session = session;
   }
 
-  protected ProfilesBackup(Collection<RulesProfile> profiles) {
+  /**
+   * for unit tests
+   */
+  ProfilesBackup(Collection<RulesProfile> profiles) {
     this.profiles = profiles;
   }
 
@@ -200,7 +203,7 @@ public class ProfilesBackup implements Backupable {
         while (reader.hasMoreChildren()) {
           reader.moveDown();
           valuesRule.put(reader.getNodeName(), reader.getValue());
-          if (reader.getNodeName().equals("params")) {
+          if ("params".equals(reader.getNodeName())) {
             while (reader.hasMoreChildren()) {
               reader.moveDown();
               Map<String, String> valuesParam = readNode(reader);
index ae0ebf15c53752152c1618649cb8599249d24252..71a7a48a2c13358b4586657930ec308be4b1ead9 100644 (file)
@@ -114,6 +114,7 @@ public class FilterResult {
   }
 
   static final class StringIgnoreCaseComparator implements Comparator, Serializable {
+    private static final long serialVersionUID = 963926659201833101L;
     private int index;
 
     StringIgnoreCaseComparator(int index) {
index 856d14c98c7d5668d94475288b8692a8b9c5af94..187192bbb234c4daf882232b9d2fb96264bdcaf6 100644 (file)
@@ -44,6 +44,7 @@ import javax.servlet.http.HttpServletResponse;
 public class BatchResourcesServlet extends HttpServlet {
 
   private static final Logger LOG = LoggerFactory.getLogger(BatchResourcesServlet.class);
+  private static final long serialVersionUID = -2100128371794649028L;
 
   @Override
   public void doGet(HttpServletRequest request, HttpServletResponse response) throws ServletException, IOException {
index 83068721c6c35d0af3ac002eaf1188a0d6aa539a..afe8d4d63d38e5b1a1ebd3832d07d694a872ffd1 100644 (file)
@@ -37,6 +37,7 @@ import javax.servlet.http.HttpServletResponse;
 public class StaticResourcesServlet extends HttpServlet {
 
   private static final Logger LOG = LoggerFactory.getLogger(StaticResourcesServlet.class);
+  private static final long serialVersionUID = -2577454614650178426L;
 
   @Override
   public void doGet(HttpServletRequest request, HttpServletResponse response)
index 6f3e0c69e5266fc9473ae0c65ddb029ac494c8d4..409333632228b38a620d349f37a3ab34a7aa6685 100644 (file)
@@ -50,12 +50,12 @@ public class GwtPublisher {
     this.settings = settings;
   }
 
-  protected GwtPublisher(GwtExtension[] extensions, File outputDir) {
+  GwtPublisher(GwtExtension[] extensions, File outputDir) {
     this.extensions = extensions;
     this.outputDir = outputDir;
   }
 
-  protected GwtPublisher() {
+  GwtPublisher() {
   }
 
   public void start() {