jyapayne/Nim
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Don't report unused hints for consumed AST (#18270)

Browse source 
* Fix #18203

* Add testcase

* Fix testcase

* Fix test
This commit is contained in:
Clyybber 2021-06-16 02:45:05 +02:00 • committed by GitHub
commit c5cf21c0c4
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
2 changed files with 18 additions and 1 deletions
Download patch file Download diff file Expand all files Collapse all files

4
compiler/lookups.nim
View file

@ -364,7 +364,9 @@ proc openShadowScope*(c: PContext) =
proc closeShadowScope*(c: PContext) = proc closeShadowScope*(c: PContext) =
## closes the shadow scope, but doesn't merge any of the symbols ## closes the shadow scope, but doesn't merge any of the symbols
c.closeScope ## Does not check for unused symbols or missing forward decls since a macro
## or template consumes this AST
rawCloseScope(c)
proc mergeShadowScope*(c: PContext) = proc mergeShadowScope*(c: PContext) =
## close the existing scope and merge in all defined symbols, this will also ## close the existing scope and merge in all defined symbols, this will also

15
tests/macros/t18203.nim Normal file
View file

@ -0,0 +1,15 @@
discard """
matrix: "--hint:SuccessX:off --hint:Link:off --hint:Conf:off --hint:CC:off --hint:XDeclaredButNotUsed:on"
nimout: '''
'''
nimoutFull: true
action: compile
"""
# bug #18203
import std/macros
macro foo(x: typed) = newProc ident"bar"
proc bar() {.foo.} = raise
bar()