diff options
author | Simon Brandhof <simon.brandhof@gmail.com> | 2014-04-15 14:11:36 +0200 |
---|---|---|
committer | Simon Brandhof <simon.brandhof@gmail.com> | 2014-04-15 14:16:24 +0200 |
commit | aecea1b85e37f20794e6cddf3e9bc4697041e762 (patch) | |
tree | 91ce7c10c6369c230334203ff0f8efc9383282c2 | |
parent | f2e3b9cb60291aaf33d54515bdbf39f3c94022cb (diff) | |
download | sonarqube-aecea1b85e37f20794e6cddf3e9bc4697041e762.tar.gz sonarqube-aecea1b85e37f20794e6cddf3e9bc4697041e762.zip |
Fix some quality flaws
-rw-r--r-- | sonar-batch/src/main/java/org/sonar/batch/scan/filesystem/ExclusionFilters.java | 7 | ||||
-rw-r--r-- | sonar-batch/src/main/java/org/sonar/batch/scan/filesystem/FileIndexer.java | 7 |
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, |