-
Notifications
You must be signed in to change notification settings - Fork 140
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
reserved identifier violation #856
Comments
Are these Eigen library naming? |
💭 I suggest to avoid that such software components depend on undefined behaviour. |
Is it us, not making a good use of Eigen, or is Eigen somehow not following 100% the standard? |
💭 Would you like to distinguish remaining open issues better for the involved software libraries? |
Yes. |
💭 How will the chances evolve to delete misplaced underscores from affected identifiers? |
This change will probably work. Would you want to try a pr with the change?
Juan
…On Mon, Nov 1, 2021 at 11:58 AM Markus Elfring ***@***.***> wrote:
💭 How will the chances evolve to delete misplaced underscores from
affected identifiers
<https://en.cppreference.com/w/cpp/language/identifiers#In_declarations>?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
<#856 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/ACEKMIE6IGKMRG723I73ISDUJ22JDANCNFSM5HBB2G2A>
.
|
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
I would like to point out that identifiers like “
_Tp
” and “_MatrixType
” do eventually not fit to the expected naming convention of the C++ language standard.Would you like to adjust your selection for unique names?
The text was updated successfully, but these errors were encountered: