diff options
author | Julien HENRY <julien.henry@sonarsource.com> | 2014-08-28 18:35:08 +0200 |
---|---|---|
committer | Julien HENRY <julien.henry@sonarsource.com> | 2014-08-28 18:35:25 +0200 |
commit | 906a5e3ee07d3d36b09798fd8894e314633e293b (patch) | |
tree | 6f7e86e0bf5bde990c9b92ca5d498c6cc0fb814b /sonar-batch | |
parent | e1e54e5839138129dca33eea3f8d2e781fceab34 (diff) | |
download | sonarqube-906a5e3ee07d3d36b09798fd8894e314633e293b.tar.gz sonarqube-906a5e3ee07d3d36b09798fd8894e314633e293b.zip |
Fix some quality flaws
Diffstat (limited to 'sonar-batch')
-rw-r--r-- | sonar-batch/src/main/java/org/sonar/batch/highlighting/SyntaxHighlightingDataValueCoder.java | 3 | ||||
-rw-r--r-- | sonar-batch/src/main/java/org/sonar/batch/mediumtest/BatchMediumTester.java | 7 |
2 files changed, 5 insertions, 5 deletions
diff --git a/sonar-batch/src/main/java/org/sonar/batch/highlighting/SyntaxHighlightingDataValueCoder.java b/sonar-batch/src/main/java/org/sonar/batch/highlighting/SyntaxHighlightingDataValueCoder.java index b3be41fa2f6..621c1e5c2f5 100644 --- a/sonar-batch/src/main/java/org/sonar/batch/highlighting/SyntaxHighlightingDataValueCoder.java +++ b/sonar-batch/src/main/java/org/sonar/batch/highlighting/SyntaxHighlightingDataValueCoder.java @@ -24,6 +24,7 @@ import com.persistit.encoding.CoderContext; import com.persistit.encoding.ValueCoder; import java.util.ArrayList; +import java.util.List; public class SyntaxHighlightingDataValueCoder implements ValueCoder { @@ -41,7 +42,7 @@ public class SyntaxHighlightingDataValueCoder implements ValueCoder { @Override public Object get(Value value, Class clazz, CoderContext context) { int count = value.getInt(); - ArrayList<SyntaxHighlightingRule> rules = new ArrayList<SyntaxHighlightingRule>(count); + List<SyntaxHighlightingRule> rules = new ArrayList<SyntaxHighlightingRule>(count); for (int i = 0; i < count; i++) { rules.add((SyntaxHighlightingRule) rulesCoder.get(value, SyntaxHighlightingRule.class, context)); } diff --git a/sonar-batch/src/main/java/org/sonar/batch/mediumtest/BatchMediumTester.java b/sonar-batch/src/main/java/org/sonar/batch/mediumtest/BatchMediumTester.java index bdb052ddc1e..be3185f7b6d 100644 --- a/sonar-batch/src/main/java/org/sonar/batch/mediumtest/BatchMediumTester.java +++ b/sonar-batch/src/main/java/org/sonar/batch/mediumtest/BatchMediumTester.java @@ -19,8 +19,6 @@ */ package org.sonar.batch.mediumtest; -import org.sonar.api.batch.sensor.highlighting.TypeOfText; - import org.apache.commons.io.IOUtils; import org.slf4j.Logger; import org.slf4j.LoggerFactory; @@ -32,6 +30,7 @@ import org.sonar.api.batch.fs.InputFile; import org.sonar.api.batch.fs.InputPath; import org.sonar.api.batch.fs.internal.DefaultInputFile; import org.sonar.api.batch.sensor.duplication.DuplicationGroup; +import org.sonar.api.batch.sensor.highlighting.TypeOfText; import org.sonar.api.batch.sensor.issue.Issue; import org.sonar.api.batch.sensor.measure.Measure; import org.sonar.api.batch.sensor.symbol.Symbol; @@ -67,6 +66,7 @@ import javax.annotation.CheckForNull; import java.io.File; import java.io.FileReader; import java.util.ArrayList; +import java.util.Collections; import java.util.Date; import java.util.HashMap; import java.util.List; @@ -291,11 +291,10 @@ public class BatchMediumTester { * Get highlighting types at a given position in an inputfile * @param charIndex 0-based offset in file */ - @CheckForNull public List<TypeOfText> highlightingTypeFor(InputFile file, int charIndex) { SyntaxHighlightingData syntaxHighlightingData = highlightingPerFile.get(file); if (syntaxHighlightingData == null) { - return null; + return Collections.emptyList(); } List<TypeOfText> result = new ArrayList<TypeOfText>(); for (SyntaxHighlightingRule sortedRule : syntaxHighlightingData.syntaxHighlightingRuleSet()) { |