-
Notifications
You must be signed in to change notification settings - Fork 144
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
Remove redundant macro definition #2925
Conversation
@zhanghe9702 I think the title of this PR is wrong - you're not fixing a typo, but you are removing a redundant macro definition. it helps us when reviewing since we can quickly know what a PR is about |
(similarly you should change your commit message to remove the "Fix typo" title) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
the changes look good! thank you for noticing it and fixing it :)
@zhanghe9702 do you want some help with getting this merged? |
sorry, i didn't noticed your message , what's your suggestion? i think this just an kind of |
Something like "Remove redundant macro definition" would be good :) and then in your commit message use that for the title as well, and in the Changelog explain that you're removing the |
i have changed the title, and rebase commit |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great, thank you :D
gcc/rust/ChangeLog: * backend/rust-tree.h: removing the CLASSTYPE_VBASECLASSES macro which is duplicated three times. Signed-off-by: Zhang He <zhanghe9702@163.com>
gcc/rust/ChangeLog:
* backend/rust-tree.h: remove redundant macro definition.