]> source.dussan.org Git - sonarqube.git/commitdiff
Fix violations
authorEvgeny Mandrikov <mandrikov@gmail.com>
Sat, 28 May 2011 21:24:07 +0000 (01:24 +0400)
committerEvgeny Mandrikov <mandrikov@gmail.com>
Sat, 28 May 2011 23:09:39 +0000 (03:09 +0400)
15 files changed:
plugins/sonar-core-plugin/src/main/java/org/sonar/plugins/core/timemachine/ViolationPersisterDecorator.java
plugins/sonar-squid-java-plugin/src/main/java/org/sonar/plugins/squid/decorators/ChidamberKemererDistributionBuilder.java
plugins/sonar-squid-java-plugin/src/main/java/org/sonar/plugins/squid/decorators/FunctionComplexityDistributionBuilder.java
sonar-channel/src/main/java/org/sonar/channel/CodeBuffer.java
sonar-channel/src/main/java/org/sonar/channel/CodeReader.java
sonar-colorizer/src/main/java/org/sonar/colorizer/InlineDocTokenizer.java
sonar-plugin-api/src/main/java/org/sonar/api/batch/DecoratorContext.java
sonar-plugin-api/src/main/java/org/sonar/api/batch/InstantiationStrategy.java
sonar-plugin-api/src/main/java/org/sonar/api/batch/events/DecoratorExecutionHandler.java
sonar-plugin-api/src/main/java/org/sonar/api/batch/events/DecoratorsPhaseHandler.java
sonar-plugin-api/src/main/java/org/sonar/api/batch/events/ProjectAnalysisHandler.java
sonar-plugin-api/src/main/java/org/sonar/api/batch/events/SensorExecutionHandler.java
sonar-plugin-api/src/main/java/org/sonar/api/batch/events/SensorsPhaseHandler.java
sonar-squid/src/main/java/org/sonar/squid/Squid.java
sonar-squid/src/main/java/org/sonar/squid/indexer/QueryByMeasure.java

index 4935390e21d780d45c4b8fa3ad933f5090e6cbcc..0c011a057260586c8bec33a90d12736c087068d8 100644 (file)
@@ -25,7 +25,6 @@ import com.google.common.collect.Multimap;
 import org.apache.commons.codec.digest.DigestUtils;
 import org.apache.commons.lang.ObjectUtils;
 import org.apache.commons.lang.StringUtils;
-import org.apache.commons.lang.math.NumberUtils;
 import org.sonar.api.batch.*;
 import org.sonar.api.database.model.RuleFailureModel;
 import org.sonar.api.database.model.SnapshotSource;
index 0dac2438e659f8184f6f7583a39d0e67db054263..202395145e93d5a57961f649ef150425757386a7 100644 (file)
@@ -34,8 +34,8 @@ import org.sonar.api.resources.Scopes;
 
 public final class ChidamberKemererDistributionBuilder implements Decorator {
 
-  public static final Integer[] LCOM4_LIMITS = {2, 3, 4, 5, 10};// 1 is excluded
-  public static final Integer[] RFC_LIMITS = {0, 5, 10, 20, 30, 50, 90, 150};
+  private static final Integer[] LCOM4_LIMITS = { 2, 3, 4, 5, 10 }; // 1 is excluded
+  private static final Integer[] RFC_LIMITS = { 0, 5, 10, 20, 30, 50, 90, 150 };
 
   @DependedUpon
   public Metric generatesLcom4Distribution() {
index 6b9477ba720a714c41ba81cac0fe09e914f343c2..e60ccad8d6eb5e9a9b402fab88a9ba7d251fe429 100644 (file)
@@ -35,7 +35,7 @@ import org.sonar.java.api.JavaMethod;
  */
 public final class FunctionComplexityDistributionBuilder implements Decorator {
 
-  public static final Number[] LIMITS = {1, 2, 4, 6, 8, 10, 12};
+  private static final Number[] LIMITS = { 1, 2, 4, 6, 8, 10, 12 };
 
   @DependsUpon
   public Metric dependOnComplexity() {
@@ -52,7 +52,7 @@ public final class FunctionComplexityDistributionBuilder implements Decorator {
       RangeDistributionBuilder builder = new RangeDistributionBuilder(CoreMetrics.FUNCTION_COMPLEXITY_DISTRIBUTION, LIMITS);
       for (DecoratorContext childContext : context.getChildren()) {
         if (childContext.getResource() instanceof JavaMethod) {
-          JavaMethod javaMethod = (JavaMethod)childContext.getResource();
+          JavaMethod javaMethod = (JavaMethod) childContext.getResource();
           Measure complexity = childContext.getMeasure(CoreMetrics.COMPLEXITY);
           if (!javaMethod.isAccessor() && complexity != null) {
             builder.add(complexity.getValue());
@@ -73,4 +73,3 @@ public final class FunctionComplexityDistributionBuilder implements Decorator {
     return Java.KEY.equals(project.getLanguageKey());
   }
 }
-
index 50359838d72a5f42df576dd6bf9d5bb200e604a5..6373bb0ff40595072b873698e1744d18c4950a30 100644 (file)
@@ -62,7 +62,7 @@ public class CodeBuffer implements CharSequence {
     buffer = new char[bufferCapacity];
     Reader reader = initialCodeReader;
     for (CodeReaderFilter<?> codeReaderFilter : configuration.getCodeReaderFilters()) {
-      reader = new Filter(reader, codeReaderFilter);
+      reader = new Filter(reader, codeReaderFilter, configuration);
     }
     this.code = reader;
     fillBuffer();
@@ -240,7 +240,7 @@ public class CodeBuffer implements CharSequence {
   }
 
   /**
-   * Warning : this method returns Integer.MAX_VALUE when the buffer is fully used 
+   * Warning : this method returns Integer.MAX_VALUE when the buffer is fully used
    * as the length of the stream can't be known before having consumed all characters.
    * 
    * Integer.MAX_VALUE is returned to prevent regular expression matchers to stop consuming the stream of characters (see
@@ -289,14 +289,14 @@ public class CodeBuffer implements CharSequence {
   /**
    * Bridge class between CodeBuffer and CodeReaderFilter
    */
-  final class Filter extends FilterReader {
+  static final class Filter extends FilterReader {
 
     private CodeReaderFilter<?> codeReaderFilter;
 
-    public Filter(Reader in, CodeReaderFilter<?> codeReaderFilter) {
+    public Filter(Reader in, CodeReaderFilter<?> codeReaderFilter, CodeReaderConfiguration configuration) {
       super(in);
       this.codeReaderFilter = codeReaderFilter;
-      this.codeReaderFilter.setConfiguration(CodeBuffer.this.configuration.cloneWithoutCodeReaderFilters());
+      this.codeReaderFilter.setConfiguration(configuration.cloneWithoutCodeReaderFilters());
       this.codeReaderFilter.setReader(in);
     }
 
index 8d8673edbc121d3d7c43716913d04cac456af3e1..1033ad20668dd42a27692f8ffbb2d5cd7bb07595 100644 (file)
@@ -79,7 +79,7 @@ public class CodeReader extends CodeBuffer {
     try {
       appendable.append((char) pop());
     } catch (IOException e) {
-      throw new ChannelException(e.getMessage());
+      throw new ChannelException(e.getMessage(), e);
     }
   }
 
index 0d6d70fa4813ada94b2cace1b00313b1e00fc5c1..76f1025d71219ec4f89bc78900aed00f83eebcc3 100644 (file)
  */
 package org.sonar.colorizer;
 
-import java.util.Arrays;
-
 import org.sonar.channel.CodeReader;
 import org.sonar.channel.EndMatcher;
 
+import java.util.Arrays;
+
 public abstract class InlineDocTokenizer extends Tokenizer {
 
   private final String tagBefore;
@@ -40,12 +40,7 @@ public abstract class InlineDocTokenizer extends Tokenizer {
   public boolean consume(CodeReader code, HtmlCodeBuilder codeBuilder) {
     if (code.peek() == startToken[0] && Arrays.equals(code.peek(startToken.length), startToken)) {
       codeBuilder.appendWithoutTransforming(tagBefore);
-      code.popTo(new EndMatcher() {
-
-        public boolean match(int endFlag) {
-          return endFlag == '\r' || endFlag == '\n';
-        }
-      }, codeBuilder);
+      code.popTo(LINE_END_MATCHER, codeBuilder);
       codeBuilder.appendWithoutTransforming(tagAfter);
       return true;
     } else {
@@ -53,4 +48,10 @@ public abstract class InlineDocTokenizer extends Tokenizer {
     }
   }
 
+  private static final EndMatcher LINE_END_MATCHER = new EndMatcher() {
+    public boolean match(int endFlag) {
+      return endFlag == '\r' || endFlag == '\n';
+    }
+  };
+
 }
index 2a737ad13adfcd744d9d35eff9831e767016bc92..771a1f42ebfc824836a61ae463bbca1cee5479a8 100644 (file)
@@ -109,7 +109,7 @@ public interface DecoratorContext {
    *          the request parameters specified as a {@link ViolationQuery}
    * @return the list of violations that match those parameters
    */
-  abstract List<Violation> getViolations(ViolationQuery violationQuery);
+  List<Violation> getViolations(ViolationQuery violationQuery);
 
   /**
    * Returns all the active (= non switched-off) violations found on the current resource.
index b270d9ce0d9920e9db7cc7127aaba17e732d7afe..308fd4357ac40bb7c74c420e2f8b73fe6055963a 100644 (file)
@@ -35,12 +35,12 @@ public @interface InstantiationStrategy {
   /**
    * Shared extension. Lifecycle is the full analysis.
    */
-  public static final String PER_BATCH = "PER_BATCH";
+  String PER_BATCH = "PER_BATCH";
 
   /**
    * Created and initialized for each project and sub-project (a project is a module in Maven terminology).
    */
-  public static final String PER_PROJECT = "PER_PROJECT";
+  String PER_PROJECT = "PER_PROJECT";
 
   String value();
 }
index 0db117a37de35f8bc411f4e18963822ae855562b..74645acf36ede82b197d51dec21d77810da37746 100644 (file)
@@ -29,7 +29,7 @@ public interface DecoratorExecutionHandler extends EventHandler {
   /**
    * This interface is not intended to be implemented by clients.
    */
-  public interface DecoratorExecutionEvent {
+  interface DecoratorExecutionEvent {
 
     Decorator getDecorator();
 
index c6a218078cbb0a2934b2d764c74e5c1f82fe586c..b6a28d317b58365d588c11a19516316ae216563d 100644 (file)
@@ -31,7 +31,7 @@ public interface DecoratorsPhaseHandler extends EventHandler {
   /**
    * This interface is not intended to be implemented by clients.
    */
-  public interface DecoratorsPhaseEvent {
+  interface DecoratorsPhaseEvent {
 
     /**
      * @return list of Decorators in the order of execution
index 110a4b5f475e480a5be7d48b81231587e8c56a51..2e0de22daae245a61b28426ce27179da2104fa41 100644 (file)
@@ -29,7 +29,7 @@ public interface ProjectAnalysisHandler extends EventHandler {
   /**
    * This interface is not intended to be implemented by clients.
    */
-  public interface ProjectAnalysisEvent {
+  interface ProjectAnalysisEvent {
 
     Project getProject();
 
index 26d78596d98deaafb504e0449e5cac39cb1af75c..7ff0a38dd0240f211d19bccf0138c5eb45cfb951 100644 (file)
@@ -29,7 +29,7 @@ public interface SensorExecutionHandler extends EventHandler {
   /**
    * This interface is not intended to be implemented by clients.
    */
-  public interface SensorExecutionEvent {
+  interface SensorExecutionEvent {
 
     Sensor getSensor();
 
index 8a54de8b44284ee0844a4d29126aabf4ca113281..f368e588992a0533ffedf629790346a1c97762e9 100644 (file)
@@ -31,7 +31,7 @@ public interface SensorsPhaseHandler extends EventHandler {
   /**
    * This interface is not intended to be implemented by clients.
    */
-  public interface SensorsPhaseEvent {
+  interface SensorsPhaseEvent {
 
     /**
      * @return list of Sensors in the order of execution
index 1702f6f23dd81d8498bfbd6047561cfe9393ee5a..b2153e25ef0cadd185af7306837dd8fccc5e06b1 100644 (file)
 
 package org.sonar.squid;
 
-import java.util.Collection;
-import java.util.HashSet;
-import java.util.List;
-import java.util.Set;
-
 import org.picocontainer.MutablePicoContainer;
 import org.picocontainer.containers.TransientPicoContainer;
 import org.sonar.graph.DirectedGraph;
 import org.sonar.graph.DirectedGraphAccessor;
-import org.sonar.squid.api.CodeScanner;
-import org.sonar.squid.api.CodeVisitor;
-import org.sonar.squid.api.Query;
-import org.sonar.squid.api.SourceCode;
-import org.sonar.squid.api.SourceCodeEdge;
-import org.sonar.squid.api.SourceCodeSearchEngine;
-import org.sonar.squid.api.SourceCodeTreeDecorator;
-import org.sonar.squid.api.SourceProject;
-import org.sonar.squid.api.SquidConfiguration;
+import org.sonar.squid.api.*;
 import org.sonar.squid.indexer.SquidIndex;
 import org.sonar.squid.measures.Metric;
 import org.sonar.squid.measures.MetricDef;
 
+import java.util.Collection;
+import java.util.HashSet;
+import java.util.List;
+import java.util.Set;
+
 public class Squid implements DirectedGraphAccessor<SourceCode, SourceCodeEdge>, SourceCodeSearchEngine {
 
   private MutablePicoContainer pico;
@@ -79,7 +71,7 @@ public class Squid implements DirectedGraphAccessor<SourceCode, SourceCodeEdge>,
     SCANNER scanner = pico.getComponent(scannerClass);
     for (Object clazz : scanner.getVisitorClasses()) {
       addToPicocontainer((Class) clazz);
-      scanner.accept(pico.<CodeVisitor>getComponent((Class) clazz));
+      scanner.accept(pico.<CodeVisitor> getComponent((Class) clazz));
     }
     for (CodeVisitor externalVisitor : externalCodeVisitors) {
       scanner.accept(externalVisitor);
@@ -87,6 +79,9 @@ public class Squid implements DirectedGraphAccessor<SourceCode, SourceCodeEdge>,
     return scanner;
   }
 
+  /**
+   * @deprecated use {@link #decorateSourceCodeTreeWith(MetricDef...)} instead
+   */
   @Deprecated
   public SourceProject aggregate() {
     return decorateSourceCodeTreeWith(Metric.values());
index 4afdc7196e6965e499fe807cedbd71bab49f9547..cdc74381c2a495b477799bf1d373484efc146002 100644 (file)
@@ -34,10 +34,13 @@ public class QueryByMeasure implements Query {
   public enum Operator {
     GREATER_THAN, EQUALS, GREATER_THAN_EQUALS, LESS_THAN, LESS_THAN_EQUALS
   }
-  
+
+  /**
+   * @deprecated use {@link #QueryByMeasure(MetricDef, Operator, double)} instead
+   */
   @Deprecated
   public QueryByMeasure(Metric metric, Operator operator, double value) {
-    this((MetricDef)metric, operator, value);
+    this((MetricDef) metric, operator, value);
   }
 
   public QueryByMeasure(MetricDef metric, Operator operator, double value) {