Skip to content

[diff.cpp03.locale] Fix title of text processing library - #9203

Merged
jensmaurer merged 1 commit into
cplusplus:mainfrom
Mick235711:reorder-annex-c
Sep 20, 2026
Merged

jensmaurer merged 1 commit into
cplusplus:mainfrom
Mick235711:reorder-annex-c

Conversation

@Mick235711

Copy link
Copy Markdown
Contributor

After clause renumbering efforts in C++23 and C++26, currently [strings] is Clause 27, which makes the Annex C entries unordered in relation to actual clause numbers (for instance, under [diff.cpp20], [diff.cpp20.strings] refers to Clause 27, while [diff.cpp20.containers] refers to Clause 23; however, the former comes first). Therefore, this PR seeks to correct the situation by reordering the entries with regard to current clause numbers:

  • Swap the positions of [diff.cpp20.strings] and [diff.cpp20.containers]
  • Swap the positions of [diff.cpp14.strings] and [diff.cpp14.containers]
  • Move [diff.cpp03.strings] after [diff.cpp03.algorithms]
  • [diff.cpp03.locale] is currently the only Annex C entry that does not refer to a whole clause, making its title "28.3: localization library" which is inconsistent with everything else. This PR thus makes it refer to Clause 28 (text processing library) instead, and moves it before [diff.cpp03.numerics] to maintain the numerical ordering
    • It seems inappropriate to change the stable labels for this; there have been precedents for [diff.cppXX.foo] not referring to [foo] before (e.g. [diff.cpp23.io], [diff.stat])

@jensmaurer jensmaurer added the P2-Bug Presentational errors and omissions label Jul 18, 2026
@jensmaurer

Copy link
Copy Markdown
Member

I'm generally in favor here. However, as usual with such large text rearrangements, it's hard to review them ("did the submitter change a character or two while moving the text around?") This is not about bad faith, but about accidents.

@tkoeppe , do you want to apply as-is, do the moves yourself, or have me do the moves?

@tkoeppe

tkoeppe commented Jul 18, 2026

Copy link
Copy Markdown
Contributor

@jensmaurer If you want to do it all in one go and merge, yes, please go ahead, that'd be nice!

@Mick235711

Copy link
Copy Markdown
Contributor Author

Gentle ping - any chance of progress on this?

@jensmaurer

Copy link
Copy Markdown
Member

@Mick235711 , I've done all the clause moves. I have not done the 28.3 title fix. Please either repurpose this pull request (after rebasing) or make a new one that does just the 28.3 title fix.

@Mick235711 Mick235711 changed the title [diff] Reorder Annex C entries to respect clause numbering [diff.cpp03.locale] Fix title of text processing library Sep 20, 2026
@Mick235711

Copy link
Copy Markdown
Contributor Author

Thanks a lot. I have repurposed this PR to only do a single renaming, and replaced the commit and PR title.

@jensmaurer
jensmaurer merged commit 4c8147d into cplusplus:main Sep 20, 2026
2 checks passed
@Mick235711
Mick235711 deleted the reorder-annex-c branch September 21, 2026 05:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

P2-Bug Presentational errors and omissions

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants