-
-
Notifications
You must be signed in to change notification settings - Fork 394
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
refactor(ast_codegen): use doc comments instead of insert!
#4777
refactor(ast_codegen): use doc comments instead of insert!
#4777
Conversation
Your org has enabled the Graphite merge queue for merging into mainAdd the label “merge” to the PR and Graphite will automatically add it to the merge queue when it’s ready to merge. Or use the label “hotfix” to add to the merge queue as a hot fix. You must have a Graphite account and log in to Graphite in order to use the merge queue. Sign up using this link. |
@rzvxa The |
CodSpeed Performance ReportMerging #4777 will not alter performanceComparing Summary
|
1266d4d
to
09d8aec
Compare
d5f689c
to
ee406fa
Compare
To be honest I don't like this change, It is too complicated for a simple thing. With that said, Feel free to merge as you wish. |
09d8aec
to
3136858
Compare
ee406fa
to
7713818
Compare
3136858
to
0b9a2eb
Compare
7713818
to
ef525ec
Compare
ef525ec
to
bb2cd39
Compare
0b9a2eb
to
c15c931
Compare
bb2cd39
to
1605fa1
Compare
OK. My feeling is that it's worth it for making generators/visit.rs clearer. Since you've given me permission to merge, I'm going to do that. |
Merge activity
|
Avoid the `insert!` macro in AST codegen. Use doc comments starting with special symbol `@` instead. * Before: `insert!("// plain comment");` * After: `///@ plain comment` * Or: `//!@ plain comment` Either `///@` or `//!@` is converted to plain `//` in output. `//!@` is legal in top-of-file position, which allows us to inline `#![allow(...)]` attributes, which in my opinion makes the generators a bit easier to read.
1605fa1
to
f418f62
Compare
Avoid the
insert!
macro in AST codegen. Use doc comments starting with special symbol@
instead.insert!("// plain comment");
///@ plain comment
//!@ plain comment
Either
///@
or//!@
is converted to plain//
in output.//!@
is legal in top-of-file position, which allows us to inline#![allow(...)]
attributes, which in my opinion makes the generators a bit easier to read.