Sonar

Support the annotation "@SuppressWarnings" at class and method level

Details

  • Type: New Feature New Feature
  • Status: Closed Closed
  • Priority: Major Major
  • Resolution: Fixed
  • Affects Version/s: None
  • Fix Version/s: 2.5
  • Component/s: Rules, Squid
  • Labels:
    None
  • Number of attachments :
    0

Description

The //NOSONAR tag is useful to deactivate all rules at a given line but is not suitable to deactivate all rules (or only a given rule) for all the lines of a method or a class.

In that case, the use of the @SuppressWarnings would perfectly fit the needs.

The excepted value to deactivate all rules would be :

@SuppressWarnings("all") //which is pretty standard

The excepted value to deactivate only one rule or a list of rules would be :

@SuppressWarnings("MY_RULE_KEY")
@SuppressWarnings({"MY_RULE_KEY1","MY_RULE_KEY2})

Issue Links

is depended upon by

New Feature - A new feature of the product, which has yet to be developed. SONARJAVA-57 Add rule key as parameters of the Java annotation @SuppressWarnings

  • Major - Major loss of function.
  • Open - The issue is open and ready for the assignee to start work on it.

Improvement - An improvement or enhancement to an existing feature or task. SONARJAVA-76 Add an option to automatically exclude "@deprecated" method from Sonar analysis (violations, lcom4, ...)

  • Major - Major loss of function.
  • Open - The issue is open and ready for the assignee to start work on it.

Wish - General wishlist item SONARJAVA-71 Support for @Generated annotation

  • Major - Major loss of function.
  • Open - The issue is open and ready for the assignee to start work on it.
is duplicated by

Improvement - An improvement or enhancement to an existing feature or task. SONAR-1407 Apply NOSONAR to a block of code

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

Wish - General wishlist item SONAR-1796 Manage an Annotation @SonarRuleIgnore to avoid checking rules on particular lines

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

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.

Improvement - An improvement or enhancement to an existing feature or task. SONARJAVA-78 UnusedPrivateMethod should read @SuppressWarnings("unused") metadata

  • Major - Major loss of function.
  • Open - The issue is open and ready for the assignee to start work on it.

Activity

Ascending order - Click to sort in descending order
Hide
Permalink
Evgeny Mandrikov added a comment -

Together with Freddy we decided that as first step we must support only 

@SuppressWarnings("all")

, so done in r6048.

Also note that this annotation disables all violations, even produced by Checkstyle rule SuppressWarnings, but not new Squid rule (SONAR-1817). So maybe we should create analogue of this rule, but for annotation.

Show
Evgeny Mandrikov added a comment - Together with Freddy we decided that as first step we must support only @SuppressWarnings( "all" ) , so done in r6048 . Also note that this annotation disables all violations, even produced by Checkstyle rule SuppressWarnings, but not new Squid rule ( SONAR-1817 ). So maybe we should create analogue of this rule, but for annotation.
Hide
Permalink
Evgeny Mandrikov added a comment -

Fixed for class level in r6381.

Show
Evgeny Mandrikov added a comment - Fixed for class level in r6381 .

People

  • Assignee:
    Freddy Mallet
    Reporter:
    Freddy Mallet
Vote (4)
Watch (4)

Dates

  • Created:
    Updated:
    Resolved: