iframe-proxy | Sunbelt Computer Software

40 lines (36 loc) · 1.05 KB

 * @name Unused parameter
 * @description Parameter is defined but not used
 * @kind problem
 * @tags quality
 *       reliability
 *       correctness
 *       readability
 * @problem.severity recommendation
 * @sub-severity high
 * @precision medium
 * @id py/unused-parameter
import python
import Definition
private import LegacyPointsTo
predicate unused_parameter(FunctionValue f, LocalVariable v) {
  v.isParameter() and
  v.getScope() = f.getScope() and
  not name_acceptable_for_unused_variable(v) and
  not exists(NameNode u | u.uses(v)) and
  not exists(Name inner, LocalVariable iv |
    inner.uses(iv) and iv.getId() = v.getId() and inner.getScope().getScope() = v.getScope()
predicate is_abstract(FunctionValue func) {
  func.getScope().getADecorator().(Name).getId().matches("%abstract%")
from PythonFunctionValue f, LocalVariable v
  v.getId() != "self" and
  unused_parameter(f, v) and
  not f.isOverridingMethod() and
  not f.isOverriddenMethod() and
  not is_abstract(f)
select f, "The parameter '" + v.getId() + "' is never used."

Sunbelt Computer Software

PL/B Language Development and Support

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

Sunbelt Computer Software

PL/B Language Development and Support

Uh oh!

FilesExpand file tree

UnusedParameter.ql

Latest commit

History

UnusedParameter.ql

File metadata and controls