Sonar

The Squid "Unused private method" doesn't report a violation at correct line with generic methods

Details

  • Type: Bug Bug
  • 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

For instance with the following java class, the violation is reported at line 0 instead of line 6 :

package org.larrys;

import java.util.List;

public class PrivateMethod {
  private void transactionValidation(String transaction, List<String> containers) {
  }
}

Issue Links

relates to

Bug - A problem which impairs or prevents the functions of the product. SONARPLUGINS-799 NPE when getting line id from Violation

  • 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
Freddy Mallet added a comment -

In fact, this case is correctly handled : A unit test already exists and some examples exist on Nemo : http://nemo.sonarsource.org/resource/index/org.sigmah:sigmah:org.sigmah.shared.report.content.PivotTableData?metric=violations

So there must be another issue.

Show
Freddy Mallet added a comment - In fact, this case is correctly handled : A unit test already exists and some examples exist on Nemo : http://nemo.sonarsource.org/resource/index/org.sigmah:sigmah:org.sigmah.shared.report.content.PivotTableData?metric=violations So there must be another issue.
Hide
Permalink
Freddy Mallet added a comment -

The problem has been finally reproduced and seems to come from the "org.sonar.java.signature.MethodSignatureScanner"

Show
Freddy Mallet added a comment - The problem has been finally reproduced and seems to come from the "org.sonar.java.signature.MethodSignatureScanner"

People

  • Assignee:
    Olivier Gaudin
    Reporter:
    Freddy Mallet
Vote (0)
Watch (0)

Dates

  • Created:
    Updated:
    Resolved: