-
Notifications
You must be signed in to change notification settings - Fork 97
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Is the "Macros in the AST" section still up to date? #19
Comments
Doesn't seem up to date. Seems like a vestige of the days before user defined proc macros were possible. |
I think this is outdated too, right?
Also imo it is contradicted by this paragraph below:
To me it seems, the chapter would be more accurate if these two sentences above were removed completely. Should I make a PR? |
Macro processing does happen after construction of the AST. There is just a special AST node for macro calls that contains a token tree. No changes have been merged for five years now. |
The macros in the AST section says:
(emphasis mine)
But a custom-derive macro can use these forms, right? (otherwise sorry for the noise).
The text was updated successfully, but these errors were encountered: