[Protoc] C++ Resolved an issue where NO_DESTROY and CONSTINIT are in incorrect order#8296
Conversation
|
Thanks for your pull request. It looks like this may be your first contribution to a Google open source project (if not, look below for help). Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA). 📝 Please visit https://cla.developers.google.com/ to sign. Once you've signed (or fixed any issues), please reply here with What to do if you already signed the CLAIndividual signers
Corporate signers
ℹ️ Googlers: Go here for more info. |
@googlebot I signed it! |
|
@Yuhanun There are also a large number of fixable instances of |
|
see: mumbleskates@30d51ad |
I thought these were generated. I'll fix them as soon as I get to it. How do I make the tests run in the PR? |
I'm assuming this means you've fixed it?
Nevermind, it's on your own fork. |
|
Pushed these changes to my Fork. |
|
Any update on this? This issue is preventing CLANG from building. |
|
Thanks for this fix, @Yuhanun. |
|
No worries! |

According to cppreference: https://en.cppreference.com/w/cpp/language/declarations#Simple_declaration
It should be:
Attributes Specifiers Type Identifier [ = init_value];not:
Specifiers Attributes Type Identifier [ = init_value];This pull request solves that.
Issue is shown here:
https://godbolt.org/z/xncqfh