Sonar Plugins

Rule: Avoid unused function argument

Details

  • Type: New Feature New Feature
  • Status: Closed Closed
  • Priority: Major Major
  • Resolution: Fixed
  • Affects Version/s: None
  • Fix Version/s: JAVASCRIPT-1.3
  • Component/s: JavaScript
  • Labels:
    None
  • Number of attachments :
    0

Description

function fun(a, b) { // "b" unused
  print(a);
}

Issue Links

is related to

Task - A task that needs to be done. SONARPLUGINS-2743 Update the SQALE mapping for new rules

  • Major - Major loss of function.
  • 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
Evgeny Mandrikov added a comment -

Integration tests were updated.

Show
Evgeny Mandrikov added a comment - Integration tests were updated.
Hide
Permalink
Freddy Mallet added a comment -

Works well Evgeny, could you just update the description :

An unused function argument is a kind of dead code. Such unused argument should be removed to increase the maintainability of a javascript program.

The following code snippet illustrates this rule :

function fun(a, b) { // Non-Compliant as 'a' argument is not used  
  var result = b/3;
  return b;
}
Show
Freddy Mallet added a comment - Works well Evgeny, could you just update the description : An unused function argument is a kind of dead code. Such unused argument should be removed to increase the maintainability of a javascript program. The following code snippet illustrates this rule : function fun(a, b) { // Non-Compliant as 'a' argument is not used var result = b/3; return b; }
Hide
Permalink
Evgeny Mandrikov added a comment -

Done.

Show
Evgeny Mandrikov added a comment - Done.
Hide
Permalink
Freddy Mallet added a comment -

Manually tested !

Show
Freddy Mallet added a comment - Manually tested !

People

  • Assignee:
    Evgeny Mandrikov
    Reporter:
    Evgeny Mandrikov
Vote (0)
Watch (2)

Dates

  • Created:
    Updated:
    Resolved: