-
Notifications
You must be signed in to change notification settings - Fork 1.6k
Update preprocessor #else and #endif comment
#3950
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
Conversation
… omit them in single-line, non-nested contexts.
…efault)" comment.
|
I've pushed a merge with
|
|
These comment changes aren't risky; we can merge them without a second maintainer approval. |
|
I'm mirroring this to the MSVC-internal repo - please notify me if any further changes are pushed. |
|
Thanks again for improving codebase consistency! 📜 📄 ✨ |
More towards #351. Product files (non-test).
Skipped in the previous #3900
&&/||, de-morganed#endif, not specifically targeted in the previous PRyals_core.h/yvals.hmacro introduction#elif-containing blocks