summaryrefslogtreecommitdiffstats
path: root/sonar-batch
diff options
context:
space:
mode:
authorJulien HENRY <julien.henry@sonarsource.com>2014-08-28 18:35:08 +0200
committerJulien HENRY <julien.henry@sonarsource.com>2014-08-28 18:35:25 +0200
commit906a5e3ee07d3d36b09798fd8894e314633e293b (patch)
tree6f7e86e0bf5bde990c9b92ca5d498c6cc0fb814b /sonar-batch
parente1e54e5839138129dca33eea3f8d2e781fceab34 (diff)
downloadsonarqube-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.java3
-rw-r--r--sonar-batch/src/main/java/org/sonar/batch/mediumtest/BatchMediumTester.java7
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()) {