diff options
author | Simon Brandhof <simon.brandhof@gmail.com> | 2013-12-09 22:02:28 +0100 |
---|---|---|
committer | Simon Brandhof <simon.brandhof@gmail.com> | 2013-12-09 22:02:28 +0100 |
commit | f0cbff8e9697dd5cb4999eee5e6f92871eacdfe2 (patch) | |
tree | 300024d21d9653b784d0eca86d9b818631f3612e /sonar-plugin-api | |
parent | 9b8d4d064f0c719ae1a96fc2140912a90b367706 (diff) | |
download | sonarqube-f0cbff8e9697dd5cb4999eee5e6f92871eacdfe2.tar.gz sonarqube-f0cbff8e9697dd5cb4999eee5e6f92871eacdfe2.zip |
Fix some quality flaws
Diffstat (limited to 'sonar-plugin-api')
-rw-r--r-- | sonar-plugin-api/src/main/java/org/sonar/api/issue/batch/IssueFilter.java | 4 | ||||
-rw-r--r-- | sonar-plugin-api/src/main/java/org/sonar/api/issue/batch/IssueFilterChain.java | 3 |
2 files changed, 2 insertions, 5 deletions
diff --git a/sonar-plugin-api/src/main/java/org/sonar/api/issue/batch/IssueFilter.java b/sonar-plugin-api/src/main/java/org/sonar/api/issue/batch/IssueFilter.java index ac20b11c6b3..c05d4cea49a 100644 --- a/sonar-plugin-api/src/main/java/org/sonar/api/issue/batch/IssueFilter.java +++ b/sonar-plugin-api/src/main/java/org/sonar/api/issue/batch/IssueFilter.java @@ -36,10 +36,10 @@ public interface IssueFilter extends BatchExtension { * <li>Switching-off an issue based on its context (<code>//NOSONAR</code> comments, semantic annotations)</li> * </ul> * The <code>chain</code> parameter allows for fine control of the filtering logic: it is each filter's duty to either pass the issue to the next filter, by calling - * the {@link IssueFilterChain#accept()} method, or return directly if the issue has to be accepted or not + * the {@link IssueFilterChain#accept(org.sonar.api.issue.Issue)} method, or return directly if the issue has to be accepted or not * @param issue the issue being filtered * @param chain the rest of the filters - * @return <code>true</code> to accept the issue, <code>false</code> to reject it, {@link IssueFilterChain#accept()} to let the other filters decide. + * @return <code>true</code> to accept the issue, <code>false</code> to reject it, {@link IssueFilterChain#accept(org.sonar.api.issue.Issue)} to let the other filters decide. */ boolean accept(Issue issue, IssueFilterChain chain); } diff --git a/sonar-plugin-api/src/main/java/org/sonar/api/issue/batch/IssueFilterChain.java b/sonar-plugin-api/src/main/java/org/sonar/api/issue/batch/IssueFilterChain.java index b147e2761e3..bb4a81d6471 100644 --- a/sonar-plugin-api/src/main/java/org/sonar/api/issue/batch/IssueFilterChain.java +++ b/sonar-plugin-api/src/main/java/org/sonar/api/issue/batch/IssueFilterChain.java @@ -29,14 +29,11 @@ import org.sonar.api.issue.Issue; * <li>Let downstream filters decide by passing the issue to the rest of the chain</li> * </ul> * @since 4.0 - * */ public interface IssueFilterChain { /** * Called by a filter to let downstream filters decide the fate of the issue - * @param issue - * @return */ boolean accept(Issue issue); } |