aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorSimon Brandhof <simon.brandhof@gmail.com>2014-04-15 14:11:36 +0200
committerSimon Brandhof <simon.brandhof@gmail.com>2014-04-15 14:16:24 +0200
commitaecea1b85e37f20794e6cddf3e9bc4697041e762 (patch)
tree91ce7c10c6369c230334203ff0f8efc9383282c2
parentf2e3b9cb60291aaf33d54515bdbf39f3c94022cb (diff)
downloadsonarqube-aecea1b85e37f20794e6cddf3e9bc4697041e762.tar.gz
sonarqube-aecea1b85e37f20794e6cddf3e9bc4697041e762.zip
Fix some quality flaws
-rw-r--r--sonar-batch/src/main/java/org/sonar/batch/scan/filesystem/ExclusionFilters.java7
-rw-r--r--sonar-batch/src/main/java/org/sonar/batch/scan/filesystem/FileIndexer.java7
2 files changed, 7 insertions, 7 deletions
diff --git a/sonar-batch/src/main/java/org/sonar/batch/scan/filesystem/ExclusionFilters.java b/sonar-batch/src/main/java/org/sonar/batch/scan/filesystem/ExclusionFilters.java
index 7b8297ea062..ced596ed9dd 100644
--- a/sonar-batch/src/main/java/org/sonar/batch/scan/filesystem/ExclusionFilters.java
+++ b/sonar-batch/src/main/java/org/sonar/batch/scan/filesystem/ExclusionFilters.java
@@ -61,15 +61,18 @@ public class ExclusionFilters implements BatchComponent {
}
public boolean accept(InputFile inputFile, InputFile.Type type) {
- PathPattern[] inclusionPatterns = null;
- PathPattern[] exclusionPatterns = null;
+ PathPattern[] inclusionPatterns;
+ PathPattern[] exclusionPatterns;
if (InputFile.Type.MAIN == type) {
inclusionPatterns = mainInclusions;
exclusionPatterns = mainExclusions;
} else if (InputFile.Type.TEST == type) {
inclusionPatterns = testInclusions;
exclusionPatterns = testExclusions;
+ } else {
+ throw new IllegalArgumentException("Unknown file type: " + type);
}
+
if (inclusionPatterns.length > 0) {
boolean matchInclusion = false;
for (PathPattern pattern : inclusionPatterns) {
diff --git a/sonar-batch/src/main/java/org/sonar/batch/scan/filesystem/FileIndexer.java b/sonar-batch/src/main/java/org/sonar/batch/scan/filesystem/FileIndexer.java
index b4afd35d7d2..3d09990018e 100644
--- a/sonar-batch/src/main/java/org/sonar/batch/scan/filesystem/FileIndexer.java
+++ b/sonar-batch/src/main/java/org/sonar/batch/scan/filesystem/FileIndexer.java
@@ -110,12 +110,9 @@ public class FileIndexer implements BatchComponent {
Collection<File> files = FileUtils.listFiles(dirToIndex, FILE_FILTER, DIR_FILTER);
for (File file : files) {
DefaultInputFile inputFile = inputFileBuilder.create(file);
- if (inputFile != null) {
- if (exclusionFilters.accept(inputFile, type)) {
- indexFile(inputFileBuilder, fileSystem, status, inputFile, type);
- }
+ if (inputFile != null && exclusionFilters.accept(inputFile, type))
+ indexFile(inputFileBuilder, fileSystem, status, inputFile, type);
}
- }
}
private void indexFile(InputFileBuilder inputFileBuilder, DefaultModuleFileSystem fs,