improve --declaredLocs to help disambiguate types (generics, aliases etc)
#18389
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
addDeclaredLocnow handles a few more type kinds by chasing the underlying declaration (follows handle tyUserTypeClassInst in addDeclaredLoc #18236)--declaredlocswe now usepreferMixed(introduced in typeToString can now show (recursively) resolved type aliases; fixes #8569 #8083 #8570 #11678) which resolves types recursively, eg:seq[MyInt2{char}]instead ofseq[MyInt2]Example
nim r --declaredLocs --hints:off tests/errmsgs/tdeclaredlocs.nimbefore PR
after PR
future work
should be mentioned only once