search

WolframResearch / codeparser

Parse Wolfram Language source code as abstract syntax trees (ASTs) or concrete syntax trees (CSTs)
MIT License
122 stars 14 forks source link

Why do Slot and MessageName remove the Source of their children #5

Closed kenkangxgwe closed 4 years ago

kenkangxgwe commented 4 years ago

As the following lines indicate, the message name and slot number/string are removed as a result of abstractSymbolString. Is there any reason that we need to remove their Sources? https://github.com/WolframResearch/codeparser/blob/5638fb6f5bf10604f7511863f6b02c9421fa6e6f/CodeParser/Kernel/Abstract.wl#L1718 https://github.com/WolframResearch/codeparser/blob/5638fb6f5bf10604f7511863f6b02c9421fa6e6f/CodeParser/Kernel/Abstract.wl#L113

bostick commented 4 years ago

I will fix these up. Thanks!

bostick commented 4 years ago

This is addressed in this commit: https://github.com/WolframResearch/codeparser/commit/ab88a91a60ca8bbf70ce5175eba4020b74a790bc