diff options
| author | Tibor Blenessy | 2018-09-13 16:27:09 +0200 |
|---|---|---|
| committer | Elena Vilchik | 2018-09-13 16:27:09 +0200 |
| commit | 007084a29468d1ce2b3f5c772fd5adca0a5d5c4e (patch) | |
| tree | 73d939f91f9115e08b42b4a1f5457a18779dd657 /sonar-css-plugin/src | |
| parent | c457c8771bd540d6867af4673e1a14c55bf7dcc6 (diff) | |
| download | sonar-css-007084a29468d1ce2b3f5c772fd5adca0a5d5c4e.tar.bz2 | |
Dont specifically handle InterruptedException (#129)
Diffstat (limited to 'sonar-css-plugin/src')
| -rw-r--r-- | sonar-css-plugin/src/main/java/org/sonar/css/plugin/CssRuleSensor.java | 24 |
1 files changed, 9 insertions, 15 deletions
diff --git a/sonar-css-plugin/src/main/java/org/sonar/css/plugin/CssRuleSensor.java b/sonar-css-plugin/src/main/java/org/sonar/css/plugin/CssRuleSensor.java index a34b433..0e4f5fc 100644 --- a/sonar-css-plugin/src/main/java/org/sonar/css/plugin/CssRuleSensor.java +++ b/sonar-css-plugin/src/main/java/org/sonar/css/plugin/CssRuleSensor.java @@ -104,22 +104,16 @@ public class CssRuleSensor implements Sensor { } } - private boolean isSuccessful(Process process) { - try { - int exitValue = process.waitFor(); - externalProcessStreamConsumer.await(); - // exit codes 0 and 2 are expected. 0 - means no issues were found, 2 - means that at least one "error-level" rule found issue - // see https://github.com/stylelint/stylelint/blob/master/docs/user-guide/cli.md#exit-codes - boolean isSuccessful = exitValue == 0 || exitValue == 2; - if (!isSuccessful) { - LOG.error("Analysis didn't terminate normally, please verify ERROR and WARN logs above. Exit code {}", exitValue); - } - return isSuccessful; - } catch (InterruptedException e) { - LOG.warn("InterruptedException while waiting for external process to finish", e); - Thread.currentThread().interrupt(); - return false; + private boolean isSuccessful(Process process) throws InterruptedException { + int exitValue = process.waitFor(); + externalProcessStreamConsumer.await(); + // exit codes 0 and 2 are expected. 0 - means no issues were found, 2 - means that at least one "error-level" rule found issue + // see https://github.com/stylelint/stylelint/blob/master/docs/user-guide/cli.md#exit-codes + boolean isSuccessful = exitValue == 0 || exitValue == 2; + if (!isSuccessful) { + LOG.error("Analysis didn't terminate normally, please verify ERROR and WARN logs above. Exit code {}", exitValue); } + return isSuccessful; } private boolean checkCompatibleNodeVersion(SensorContext context) { |
