-
Notifications
You must be signed in to change notification settings - Fork 7
Apply Beman Standard and add more examples/docs #28
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
This compiler, like clang, doesn't properly advertise deducing this support. I really don't know the point of these feature test macros if we have to work around them not being used for almost every compiler. :(
6697035 to
f4f999a
Compare
|
@steve-downey @camio , can you take a look? |
steve-downey
left a comment
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.
👍
RaduNichita
left a comment
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.
LGTM
5d2762f to
4c9339b
Compare
|
Merging this for Meeting C++ presentation. I will do follow-up PRs. Please continue to review! |
|
LGTM! 🎉 |
|
"Contemporary" has been suggested as a replacement for "modern", since modern is a decade old style? |
Issues: #1 #4 #6