forks
/
D-Scanner
mirror of https://github.com/dlang-community/D-Scanner.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

Don't warn about renamed imports.

This commit is contained in:
dhasenan 2016-03-14 17:22:56 +00:00
parent f7868cae73
commit 673e784a03
1 changed files with 28 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

32
src/analysis/local_imports.d
View File

@ -52,10 +52,16 @@ class LocalImportCheck : BaseAnalyzer
if ((!isStatic && interesting) && (id.importBindings is null
|| id.importBindings.importBinds.length == 0))
{
addErrorMessage(id.singleImports[0].identifierChain.identifiers[0].line,
id.singleImports[0].identifierChain.identifiers[0].column, "dscanner.suspicious.local_imports",
"Local imports should specify"
~ " the symbols being imported to avoid hiding local symbols.");
foreach (singleImport; id.singleImports)
{
if (singleImport.rename.text.length == 0)
{
addErrorMessage(singleImport.identifierChain.identifiers[0].line,
singleImport.identifierChain.identifiers[0].column,
"dscanner.suspicious.local_imports", "Local imports should specify"
~ " the symbols being imported to avoid hiding local symbols.");
}
}
}
}
@ -75,3 +81,21 @@ private:
bool interesting;
bool isStatic;
}
unittest
{
import analysis.config : StaticAnalysisConfig;
StaticAnalysisConfig sac;
sac.local_import_check = true;
assertAnalyzerWarnings(q{
void testLocalImport()
{
import std.stdio; // [warn]: Local imports should specify the symbols being imported to avoid hiding local symbols.
import std.fish : scales, head;
import DAGRON = std.experimental.dragon;
}
}c, sac);
stderr.writeln("Unittest for LocalImportCheck passed.");
}