Closed zerbina closed 4 months ago
/merge
Merge requested by: @saem
Contents after the first section break of the PR description has been removed and preserved below:
## Notes for Reviewers * an interim solution until symbol binding/lookup for templates is overhauled, at which point the node flag becomes obsolete again * a regression introduced by #1298
Summary
Fix a regression where field names with backticks in them couldn't be accessed with a field access.
Fixes https://github.com/nim-works/nimskull/issues/1379.
Details
nfWasGensym
)originalName
only strips thegensym
suffix from identifiers marked with the flagLooking for the full "`gensym" suffix wouldn't work, because it would also trigger the stripping for user-created names containing the suffix.