-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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
Update preprocessor #else
and #endif
comment
#3950
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.h
macro introduction#elif
-containing blocks