diff options
| author | Hao Xia <hao@optimizely.com> | 2016-08-09 14:07:26 -0700 |
|---|---|---|
| committer | Hao Xia <hao@optimizely.com> | 2017-01-11 16:48:35 -0800 |
| commit | aa7e3359be92afbb6d3cc8dee2139a9872a03a57 (patch) | |
| tree | 2df317963f621e8cc3ebc820595ff2784bb18de1 /src/compiler/scala/tools/nsc/typechecker/Contexts.scala | |
| parent | c89d821a4f875133abeccd68d996e0a08f93e08d (diff) | |
| download | scala-aa7e3359be92afbb6d3cc8dee2139a9872a03a57.tar.gz scala-aa7e3359be92afbb6d3cc8dee2139a9872a03a57.tar.bz2 scala-aa7e3359be92afbb6d3cc8dee2139a9872a03a57.zip | |
Fix ImportHandler's reporting of importedNames and importedSymbols
Diffstat (limited to 'src/compiler/scala/tools/nsc/typechecker/Contexts.scala')
| -rw-r--r-- | src/compiler/scala/tools/nsc/typechecker/Contexts.scala | 7 |
1 files changed, 5 insertions, 2 deletions
diff --git a/src/compiler/scala/tools/nsc/typechecker/Contexts.scala b/src/compiler/scala/tools/nsc/typechecker/Contexts.scala index fde2f7bb03..2806c04d59 100644 --- a/src/compiler/scala/tools/nsc/typechecker/Contexts.scala +++ b/src/compiler/scala/tools/nsc/typechecker/Contexts.scala @@ -64,9 +64,8 @@ trait Contexts { self: Analyzer => for (imps <- allImportInfos.remove(unit)) { for (imp <- imps.reverse.distinct) { val used = allUsedSelectors(imp) - def isMask(s: ImportSelector) = s.name != nme.WILDCARD && s.rename == nme.WILDCARD - imp.tree.selectors filterNot (s => isMask(s) || used(s)) foreach { sel => + imp.tree.selectors filterNot (s => isMaskImport(s) || used(s)) foreach { sel => reporter.warning(imp posOf sel, "Unused import") } } @@ -74,6 +73,10 @@ trait Contexts { self: Analyzer => } } + def isMaskImport(s: ImportSelector): Boolean = s.name != nme.WILDCARD && s.rename == nme.WILDCARD + def isIndividualImport(s: ImportSelector): Boolean = s.name != nme.WILDCARD && s.rename != nme.WILDCARD + def isWildcardImport(s: ImportSelector): Boolean = s.name == nme.WILDCARD + var lastAccessCheckDetails: String = "" /** List of symbols to import from in a root context. Typically that |