Sonar Java

Add rule key as parameters of the Java annotation @SuppressWarnings

Details

  • Type: New Feature New Feature
  • Status: Open Open
  • Priority: Major Major
  • Resolution: Unresolved
  • Affects Version/s: None
  • Fix Version/s: None
  • Component/s: API, Java, Squid
  • Labels:
    None
  • Number of attachments :
    0

Description

The annotation @SuppressWarnings is supported since v.2.5 but it accepts only the parameter "all" to exclude any violations (see SONAR-1760).

It should accept a list of rule keys to exclude only violations relating to a subset of rules :

@SuppressWarnings("FIRST_RULE", "SECOND_RULE")

Issue Links

depends upon

New Feature - A new feature of the product, which has yet to be developed. SONAR-1760 Support the annotation "@SuppressWarnings" at class and method level

  • Major - Major loss of function.
  • Closed - The issue is considered finished, the resolution is correct. Issues which are not closed can be reopened.
is duplicated by

New Feature - A new feature of the product, which has yet to be developed. SONAR-2755 Support @SuppressWarnings("MY_RULE_KEY") and @SuppressWarnings({"MY_RULE_KEY1","MY_RULE_KEY2})

  • Critical - Crashes, loss of data, severe memory leak.
  • Closed - The issue is considered finished, the resolution is correct. Issues which are not closed can be reopened.

Activity

Ascending order - Click to sort in descending order
Hide
Permalink
Harry Callahan added a comment -

Usually there are many false errors if the analyzed project uses spring, swing gui designers, gwt etc
It is absolutely necessary to support @SuppressWarnings("MY_RULE_KEY") and @SuppressWarnings(

{"MY_RULE_KEY1","MY_RULE_KEY2}

)
annotations to be able to disable some specific errors in some places.
It is an absolutely mandatory feature.

Show
Harry Callahan added a comment - Usually there are many false errors if the analyzed project uses spring, swing gui designers, gwt etc It is absolutely necessary to support @SuppressWarnings("MY_RULE_KEY") and @SuppressWarnings( {"MY_RULE_KEY1","MY_RULE_KEY2} ) annotations to be able to disable some specific errors in some places. It is an absolutely mandatory feature.
Hide
Permalink
Leandro added a comment -

What is the fix version for this issue?

Show
Leandro added a comment - What is the fix version for this issue?
Hide
Permalink
Ketoth Xupack added a comment -

Support of this feature would be just awesome because at last it provides unified way to suppress QA warnings from different tools.
For example

  • CheckStyle do not support @SuppressWarnings annotation like PMD
  • Findbugs have it's own extended @SuppressWarnings annotation

Also I should notice that @SuppressWarnings have limitation - it can't be specified at the package level (at last until bug #6299893 will be reopened and fixed). Anyway support of either some custom sonar @SuppressWarnings annotation (like FB one) or standard one would be killer feature

Show
Ketoth Xupack added a comment - Support of this feature would be just awesome because at last it provides unified way to suppress QA warnings from different tools. For example CheckStyle do not support @SuppressWarnings annotation like PMD Findbugs have it's own extended @SuppressWarnings annotation Also I should notice that @SuppressWarnings have limitation - it can't be specified at the package level (at last until bug #6299893 will be reopened and fixed). Anyway support of either some custom sonar @SuppressWarnings annotation (like FB one) or standard one would be killer feature
Hide
Permalink
Marcel Stör added a comment -

Note that in newer versions of FindBugs annotations there's an annotation called "SuppressFBWarnings" that was introduced to avoid name clashes with the standard Java annotation: http://code.google.com/p/findbugs/source/browse/trunk/findbugs/src/java/edu/umd/cs/findbugs/annotations/SuppressFBWarnings.java

Show
Marcel Stör added a comment - Note that in newer versions of FindBugs annotations there's an annotation called "SuppressFBWarnings" that was introduced to avoid name clashes with the standard Java annotation: http://code.google.com/p/findbugs/source/browse/trunk/findbugs/src/java/edu/umd/cs/findbugs/annotations/SuppressFBWarnings.java
Hide
Permalink
Leandro added a comment -

This issue is open for 2 years now, is it really THAT hard?

Show
Leandro added a comment - This issue is open for 2 years now, is it really THAT hard?

People

  • Assignee:
    Unassigned
    Reporter:
    Simon Brandhof
Vote (17)
Watch (11)

Dates

  • Created:
    Updated: