Skip to content

Conversation

@eisenwave
Copy link
Member

This fixes two problems:

  • "cv-unqualified" is redundant here; the set of signed or unsigned integer types does not include any cv-qualified types.
  • Usually, we cross reference [basic.fundamental] when using the term "signed or unsigned integer type", and there is no obvious reason why we shouldn't also do this here, for consistency.

@eisenwave eisenwave added the P2-Bug Presentational errors and omissions label Nov 16, 2025
Copy link
Member

@jensmaurer jensmaurer left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Adding the cross-reference is fine.

We have at least one other instance of "cv-unqualified signed/unsigned integer type"; it would be better to have a separate pull request that removes "cv-unqualified" from them all.

@eisenwave eisenwave force-pushed the redundant-cv-unqualified-signed-or-unsigned branch from 397dd6e to e336dc9 Compare November 16, 2025 18:20
@eisenwave eisenwave requested a review from jensmaurer November 16, 2025 18:20
@eisenwave eisenwave added P3-Other Triaged issue not in P1 or P2 and removed P2-Bug Presentational errors and omissions labels Nov 16, 2025
@eisenwave eisenwave changed the title [numerics.c.ckdint] Remove redundant "cv-unqualified", add cross-reference [numerics.c.ckdint] Add cross-reference to [basic.fundamental] Nov 16, 2025
@jensmaurer jensmaurer merged commit 8512806 into cplusplus:main Nov 16, 2025
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

P3-Other Triaged issue not in P1 or P2

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants