Fixed bug in unused parameter check
This commit is contained in:
parent
525790aefc
commit
7a2d5f7e73
|
@ -35,6 +35,7 @@ class UnusedVariableCheck : BaseAnalyzer
|
||||||
isOverride = isOverride || (attribute.storageClass !is null &&
|
isOverride = isOverride || (attribute.storageClass !is null &&
|
||||||
attribute.storageClass.token == tok!"override");
|
attribute.storageClass.token == tok!"override");
|
||||||
declaration.accept(this);
|
declaration.accept(this);
|
||||||
|
isOverride = false;
|
||||||
}
|
}
|
||||||
|
|
||||||
override void visit(const FunctionDeclaration functionDec)
|
override void visit(const FunctionDeclaration functionDec)
|
||||||
|
@ -42,9 +43,12 @@ class UnusedVariableCheck : BaseAnalyzer
|
||||||
pushScope();
|
pushScope();
|
||||||
if (functionDec.functionBody !is null)
|
if (functionDec.functionBody !is null)
|
||||||
{
|
{
|
||||||
|
bool ias = inAggregateScope;
|
||||||
|
inAggregateScope = false;
|
||||||
if (!isOverride)
|
if (!isOverride)
|
||||||
functionDec.parameters.accept(this);
|
functionDec.parameters.accept(this);
|
||||||
functionDec.functionBody.accept(this);
|
functionDec.functionBody.accept(this);
|
||||||
|
inAggregateScope = ias;
|
||||||
}
|
}
|
||||||
popScope();
|
popScope();
|
||||||
}
|
}
|
||||||
|
|
Loading…
Reference in New Issue