Skip to content

Fix sections numbers in existing \xrefc and \IsoC #8113

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

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

frederick-vs-ja
Copy link
Contributor

@frederick-vs-ja frederick-vs-ja commented Jul 22, 2025

Currently, there are many mistakes in the section numbers in \xrefc and after \IsoC{}.

  • Some are still section numbers in C17.
  • Some are section numbers in WG14 N3220. Unfortunately, there're many "General" sections added to C23 after N3220.
    • I'm now using section numbers in WG14 N3299 as the new changes didn't seem to alternate any section number from C23.
  • Some seem to be overly updated to match WG14 N3550. These sections are for components in <stddef.h>, while in C2y a new header <stdcountof.h> is added.
  • Some are typo.

Fixes #8109.

Copy link
Contributor

@timsong-cpp timsong-cpp left a comment

Choose a reason for hiding this comment

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

I noticed a lot of these when reviewing LWG motion 3 too. I haven't gone through all the updates though.

One item that I didn't see here: [cstdarg.syn] p1.1: Cross-reference should be 6.5.3.3.

source/text.tex Outdated
@@ -13317,7 +13317,7 @@
These functions have the semantics specified in the C standard library.
\end{itemdescr}

\xrefc{7.22.7.1, 7.22.8, 7.29.6.2.1}
\xrefc{7.24.8.1, 7.24.9, 7.31.6.3.1}
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
\xrefc{7.24.8.1, 7.24.9, 7.31.6.3.1}
\xrefc{7.24.8.2, 7.24.9, 7.31.6.3.1}

I wonder if we can just remove this entire paragraph.

@frederick-vs-ja frederick-vs-ja changed the title Fix sections numbers in existing \xrefc Fix sections numbers in existing \xrefc and \IsoC Jul 22, 2025
@frederick-vs-ja
Copy link
Contributor Author

One item that I didn't see here: [cstdarg.syn] p1.1: Cross-reference should be 6.5.3.3.

Thanks! I initially didn't intent to fix section numbers after \IsoC{} in this PR. But it's perhaps nice to do so.

@tkoeppe
Copy link
Contributor

tkoeppe commented Jul 22, 2025

Please feel free to sqash all this into a single commit, and maybe put all the section references into the joint commit message.

@jwakely would you perhaps be able to have a look?

[intro.abstract] Update to refer to C23 7.23.3

[basic.fundamental] Update to refer to C23 5.3.5.3.2
... due to new sections added after WG14 N3220

[support.types] Un-update to refer to C23 7.21
In WG14 N3550, the section number of <stddef.h> components has become
7.22 due to addition of <stdcountof.h>.

[climits.syn] Update to refer to C23 5.3.5.3.2
... due to new sections added after WG14 N3220

[cfloat.syn] Update to refer to C23 5.3.5.3.3
... due to new sections added after WG14 N3220

[support.start.term] Update to refer to C23 7.24.5
... due to new sections added after WG14 N3220

[cstdarg.syn] Update to refer to C23 6.5.3.3
... due to new sections added after WG14 N3220

[c.malloc] Update to refer to C23 7.24.4

[alg.c.library] Update to refer to C23 7.24.6
... due to new sections added after WG14 N3220

[charconv.to.chars] Update to refer to C23 7.23.6.2

[charconv.from.chars] Update to refer to C23 7.24.2.6, 7.24.2.8

[cwctype.syn] Update to refer to C23 7.32

[c.mb.wcs] Update to refer to C23 7.24.8.2, 7.24.9, 7.31.6.3.1, 7.24.8
... and C23 7.30.2, 7.31.6.4, 7.31.6.5 due to new sections added after
WG14 N3220

[c.math.rand] Update to refer to C23 7.24.3

[c.math.abs] Update to refer to C23 7.12.8.3, 7.24.7.1
... partially due to new sections added after WG14 N3220

[c.math.fpclass] Update to refer to C23 7.12.4, 7.12.18
... partially due to new sections added after WG14 N3220, along with
typo fix

[iostream.objects.overview] Update to refer to C23 7.23.2

[print.fun] Update to refer to C23 7.23.2

[cstdio.syn] Update to refer to C23 7.23.3, 7.23

[depr.c.macros] Update to refer to C23 5.3.5.3.3, 7.33.6
... due to new sections added after WG14 N3220
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.

[c.math.fpclass] Typo
3 participants