fix, dont emit warn for body-less functions
This commit is contained in:
parent
cbeca3ba34
commit
e041f6e5dd
|
@ -52,7 +52,7 @@ public:
|
||||||
|
|
||||||
decl.accept(this);
|
decl.accept(this);
|
||||||
|
|
||||||
if (autoFun && !_returns[$-1])
|
if (decl.functionBody && autoFun && !_returns[$-1])
|
||||||
addErrorMessage(decl.name.line, decl.name.column, KEY, MESSAGE);
|
addErrorMessage(decl.name.line, decl.name.column, KEY, MESSAGE);
|
||||||
}
|
}
|
||||||
|
|
||||||
|
@ -149,5 +149,19 @@ unittest
|
||||||
AutoFunctionChecker.MESSAGE,
|
AutoFunctionChecker.MESSAGE,
|
||||||
), sac);
|
), sac);
|
||||||
|
|
||||||
|
assertAnalyzerWarnings(q{
|
||||||
|
auto doStuff(){} // [warn]: %s
|
||||||
|
extern(C) auto doStuff();
|
||||||
|
}c.format(
|
||||||
|
AutoFunctionChecker.MESSAGE,
|
||||||
|
), sac);
|
||||||
|
|
||||||
|
assertAnalyzerWarnings(q{
|
||||||
|
auto doStuff(){} // [warn]: %s
|
||||||
|
@disable auto doStuff();
|
||||||
|
}c.format(
|
||||||
|
AutoFunctionChecker.MESSAGE,
|
||||||
|
), sac);
|
||||||
|
|
||||||
stderr.writeln("Unittest for AutoFunctionChecker passed.");
|
stderr.writeln("Unittest for AutoFunctionChecker passed.");
|
||||||
}
|
}
|
||||||
|
|
Loading…
Reference in New Issue