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.

Sign up for GitHub

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

Jump to bottom

[basic.align] Fix non-mathematical wording of p7 #6438

Open

Eisenwave wants to merge 1 commit into cplusplus:main from Eisenwave:main-1

Contributor

Eisenwave commented Aug 6, 2023

The previous wording had some issues which this PR resolves:

The first two bullets talk about the numeric values of the alignment. The last bullet breaks this convention, which is surprising.
The proper wording is "greater", not "larger", when comparing numerical values.
The last bullet is missing a comma two separate the conditional clause from the rest of the sentence.
It is always somewhat problematic when two items are named. Using $A$ and $B$ is clearer.


          [basic.align] fix non-mathematical wording of p7

e8bf311

tkoeppe force-pushed the main branch from 89a9fb3 to 79eb180 Compare

April 19, 2024 13:54

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment