Merge pull request #11132 from jrfondren/expand-amb-identifier-output
provide more useful ambiguous-identifier error output
This commit is contained in:
commit
99ae911391
2 changed files with 6 additions and 5 deletions
|
|
@ -255,9 +255,10 @@ proc errorUseQualifier*(c: PContext; info: TLineInfo; s: PSym) =
|
|||
var candidate = initIdentIter(ti, c.importTable.symbols, s.name)
|
||||
var i = 0
|
||||
while candidate != nil:
|
||||
if i == 0: err.add " -- use "
|
||||
else: err.add " or "
|
||||
err.add candidate.owner.name.s & "." & candidate.name.s
|
||||
if i == 0: err.add " -- use one of the following:\n"
|
||||
else: err.add "\n"
|
||||
err.add " " & candidate.owner.name.s & "." & candidate.name.s
|
||||
err.add ": " & typeToString(candidate.typ)
|
||||
candidate = nextIdentIter(ti, c.importTable.symbols)
|
||||
inc i
|
||||
localError(c.config, info, errGenerated, err)
|
||||
|
|
|
|||
|
|
@ -1,6 +1,6 @@
|
|||
discard """
|
||||
errormsg: "ambiguous identifier: 'range' -- use system.range or mrange.range"
|
||||
line: 13
|
||||
errormsg: "ambiguous identifier: 'range' -- use one of the following:"
|
||||
line: "13"
|
||||
"""
|
||||
|
||||
import mrange
|
||||
|
|
|
|||
Loading…
Add table
Add a link
Reference in a new issue