Skip to content
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

[atomics] Reword table headers, removing mention of "inttypes.h" #584

Merged
merged 1 commit into from Dec 18, 2015

Conversation

tkoeppe
Copy link
Contributor

@tkoeppe tkoeppe commented Dec 11, 2015

inttypes.h is not the primary header that defines int_least8_t etc. That's cstdint. The only new thing inttypes.h adds is idivmax_t, but we don't have an atomic version of that.

@tkoeppe
Copy link
Contributor Author

tkoeppe commented Dec 16, 2015

@zygoloid: Is this change sufficiently editorial, or will this require a broader discussion?

@tkoeppe tkoeppe force-pushed the master branch 2 times, most recently from d6b7400 to cf63f0c Compare December 16, 2015 22:07
@tkoeppe tkoeppe changed the title [atomics] fix inttypes.h -> cstdint [atomics] Remove incomplete mention of "inttypes.h" Dec 16, 2015
@tkoeppe tkoeppe changed the title [atomics] Remove incomplete mention of "inttypes.h" [atomics] Reword table headrs, removing mention of "inttypes.h" Dec 17, 2015
@tkoeppe
Copy link
Contributor Author

tkoeppe commented Dec 17, 2015

@zygoloid: PTAL. I can remove the word "corresponding" if you think it's too much, or move it into the table caption (e.g. "named atomic types and their corresponding non-atomic counterparts").

@tkoeppe tkoeppe changed the title [atomics] Reword table headrs, removing mention of "inttypes.h" [atomics] Reword table headers, removing mention of "inttypes.h" Dec 17, 2015
zygoloid added a commit that referenced this pull request Dec 18, 2015
[atomics] Reword table headers, removing incorrect mention of "inttypes.h"
@zygoloid zygoloid merged commit dc277b4 into cplusplus:master Dec 18, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants