-
-
Notifications
You must be signed in to change notification settings - Fork 244
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
Added more C++11 override keywords #485
Conversation
Seems I didn't get them all last time... I used a newer clang here and I think this is all of them finally. |
Unfortunately, my confidence about these changes would be low. More in @bmribler 's wheelhouse than mine. :-) |
@bmribler if you're too busy, maybe you know of another C++ reviewer? |
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.
Sorry, I didn't see the messages... I'm not sure about the format failure though.
Great, thanks! Can these get into the 1.10.x branch too please? |
Yes, after a couple of days testing develop I’ll create pull requests to merge all recent pull requests that have been merged to develop but not to 1.12 and 1.10.
From: Sean McBride ***@***.***>
Sent: Wednesday, April 21, 2021 11:05 AM
To: HDFGroup/hdf5 ***@***.***>
Cc: Larry Knox ***@***.***>; State change ***@***.***>
Subject: Re: [HDFGroup/hdf5] Added more C++11 override keywords (#485)
Great, thanks! Can these get into the 1.10.x branch too please?
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub<#485 (comment)>, or unsubscribe<https://github.com/notifications/unsubscribe-auth/ADGMWCWIHJPFUL7ABGPFGA3TJ3ZTVANCNFSM4ZPZZZEA>.
|
Will that require a higher C++ standard then what is already in 1.12. I know we changed the develop setting - not sure we can move 1.12 setting? |
No, because |
No description provided.