[cstdint, cinttypes.syn] Reorganize <cstdint> presentation #843

tkoeppe · 2016-07-21T15:01:12Z

Demote heading "18.4.1 Header <cstdint> synopsis" to \synopsis.
Show MIN/MAX macros in synopsis.
Remove stray namespace comment.
Add See Also for the relevant C clause.
Add cross reference from <cinttypes> synopsis.

zygoloid · 2016-07-21T18:27:36Z

source/support.tex

 \end{codeblock}

 \pnum
 The header defines all types and macros the same as


I think this has changed the meaning, but I'm not sure whether the old or new wording matches the intent. We used to definitively say that the C++ header definitely defines these macros for N in 8, 16, 32, 64, and in no other cases. C says something quite different -- those macros are defined if there happen to be types of those widths, and other macros can also be defined if there are types of other widths.

Does C++ not actually allow other values of N at all?

@zygoloid: updated version pushed that lists the four sizes that we support explicitly.

@zygoloid: Ping

zygoloid · 2016-08-10T21:32:14Z

source/support.tex


-\pnum
-The header also defines numerous macros of the form:
+#define INT[8, 16, 32, 64]_MIN @\seebelow@         // optional


These did not appear to be optional before.

I do not think we can fix this editorially; we need to ask LWG what they meant.

These macros are supposed to expand to expressions of their corresponding type. Since the corresponding type is optional, surely the macro must also be optional?

But indeed the LEAST/FAST macros should absolutely not be optional, that's a mistake that I will fix at once.

I will open an LWG issue to have the question settled of whether the macros are optional.

tkoeppe · 2016-11-18T20:31:31Z

Pending resolution of LWG 2764 and LWG 2820.

jensmaurer · 2016-12-14T13:47:37Z

source/iostreams.tex

 \begin{codeblock}
-#include <cstdint>
+#include <cstdint> // see \ref{cstdint}



Is that comment aligned to one of your preferred columns?

Let me check. Preferred columns are 20, 32, 48, 24.

jensmaurer · 2016-12-14T13:48:44Z

source/support.tex


-\rSec2[cstdint.syn]{Header \tcode{<cstdint>} synopsis}
+\synopsis{Header \tcode{<cstdint>} synopsis}
 \indextext{\idxhdr{cstdint}}%


\synopsis is gone; see #566 and #1231.

OK, I'll rebase this.

Rebased, not sure how to avoid the pointless substructure.

jensmaurer · 2016-12-14T13:53:17Z

source/support.tex


+\xref ISO C~7.20.
+
 \rSec1[support.start.term]{Start and termination}


This seems a good addition that can go in ahead of time.

Done in edd2c5f.

jensmaurer · 2016-12-15T14:25:13Z

source/support.tex

+  using int16_t        = @\textit{signed integer type}@;    // optional
+  using int32_t        = @\textit{signed integer type}@;    // optional
+  using int64_t        = @\textit{signed integer type}@;    // optional



Could you please put in the whitespace fixes right away? This makes the "hard" parts easier to spot.

Sure, will do.

Reverting those changes for now, since they don't take italic correction into account, either. Once we have direction, I'll separate cosmetic and substantive change.

cubbimew · 2016-12-21T20:10:00Z

source/support.tex

+#define UINT_LEAST[8, 16, 32, 64]_MAX @\seebelow@

-plus function macros of the form:
+#define INT_FAST[8, 16, 32, 64]_MIN @\seebelow@


This seems worse: the never-defined pattern-matching language of "INT_[FAST LEAST]{8 16 32 64}_MIN" is both changed and used in what would otherwise be a preprocessor directive (rather than prose).

tkoeppe · 2018-04-02T01:02:13Z

I think this is pending some LWG issue (LWG 2820). I'll check with @mclow where we are on this.

jensmaurer · 2018-09-15T10:38:06Z

@tkoeppe, ping. It seems LWG 2820 is ready for a -lib vote and will make some of the changes in this pull request moot. I'd suggest you ask @mclow.

tkoeppe · 2019-03-15T21:31:40Z

Any day now, seriously.

tkoeppe · 2020-12-14T13:24:00Z

@mclow, @jensmaurer: This has been resolved by LWG2820 via 2020-11 Motion LWG-3. Thank you all for your continued support!

zygoloid reviewed Jul 21, 2016
View reviewed changes

tkoeppe force-pushed the inttypes branch from f2bb988 to dca236a Compare July 21, 2016 19:34

zygoloid reviewed Aug 10, 2016
View reviewed changes

tkoeppe force-pushed the inttypes branch from dca236a to 2c3260b Compare August 10, 2016 21:49

tkoeppe force-pushed the inttypes branch 2 times, most recently from b2c1c16 to 9739703 Compare September 26, 2016 19:17

tkoeppe force-pushed the inttypes branch from 9739703 to 9fa7b9c Compare November 2, 2016 10:26

zygoloid added the lwg Issue must be reviewed by LWG. label Nov 12, 2016

tkoeppe force-pushed the master branch from d2a10b4 to c1f9b30 Compare November 17, 2016 14:36

tkoeppe force-pushed the inttypes branch 2 times, most recently from 342b592 to a514c40 Compare November 18, 2016 20:29

tkoeppe force-pushed the inttypes branch from a514c40 to 3829f78 Compare November 28, 2016 00:29

jensmaurer reviewed Dec 14, 2016

View reviewed changes

tkoeppe force-pushed the inttypes branch 4 times, most recently from 5c10b40 to 9155f47 Compare December 15, 2016 01:32

jensmaurer reviewed Dec 15, 2016

View reviewed changes

jensmaurer mentioned this pull request Dec 20, 2016

[cstdint.syn] should list the macros it defines #1276

Closed

cubbimew reviewed Dec 21, 2016

View reviewed changes

tkoeppe force-pushed the inttypes branch from 9155f47 to 3691ad4 Compare February 5, 2017 00:22

tkoeppe force-pushed the inttypes branch from 3691ad4 to 189cf86 Compare February 23, 2017 00:07

tkoeppe force-pushed the master branch from 305fa75 to 1af1226 Compare March 21, 2017 01:46

tkoeppe force-pushed the inttypes branch from 189cf86 to 63191b1 Compare March 25, 2017 16:33

tkoeppe force-pushed the master branch from bfc1be3 to c92319d Compare June 18, 2017 15:57

tkoeppe force-pushed the inttypes branch 2 times, most recently from 9b73aae to 8f65e52 Compare July 31, 2017 13:45

tkoeppe force-pushed the master branch from dbd84ed to 019fc86 Compare September 28, 2017 21:52

tkoeppe force-pushed the inttypes branch from 99d5383 to cf829ce Compare October 16, 2017 21:58

tkoeppe force-pushed the inttypes branch from cf829ce to 6f1de3b Compare November 12, 2017 05:04

tkoeppe force-pushed the master branch from b97296d to 75c0adc Compare November 21, 2017 22:53

tkoeppe force-pushed the inttypes branch from 6f1de3b to 3d7ce5d Compare November 23, 2017 21:06

tkoeppe force-pushed the inttypes branch from 3d7ce5d to 7fb5bf6 Compare February 12, 2018 23:34

zygoloid force-pushed the master branch from 1d8071e to 58382cb Compare March 30, 2018 00:55

tkoeppe force-pushed the inttypes branch 3 times, most recently from c0519f4 to 900408e Compare April 2, 2018 22:09

burblebee force-pushed the master branch from bac9908 to 18c6909 Compare April 3, 2018 09:07

zygoloid force-pushed the master branch from 46b410b to 88dc8aa Compare June 21, 2018 22:15

tkoeppe force-pushed the inttypes branch from 900408e to 0c6ef6b Compare June 29, 2018 23:01

zygoloid force-pushed the master branch 2 times, most recently from e3dbfe2 to 1a21a65 Compare July 7, 2018 23:19

zygoloid force-pushed the master branch from b094703 to fb99e64 Compare March 9, 2019 01:29

tkoeppe force-pushed the inttypes branch 2 times, most recently from 2bbf3ca to 445f515 Compare November 9, 2019 13:44

tkoeppe force-pushed the inttypes branch from 445f515 to e72947f Compare March 6, 2020 22:29

tkoeppe force-pushed the inttypes branch 2 times, most recently from dfc71b8 to 8e899df Compare September 29, 2020 09:20

jensmaurer force-pushed the master branch from 1ed19b9 to 2edf19f Compare September 30, 2020 19:40

[cstdint, cinttypes.syn] Reorganize <cstdint> presentation

a9942fb

tkoeppe force-pushed the inttypes branch from 8e899df to a9942fb Compare December 14, 2020 13:22

tkoeppe closed this Dec 14, 2020

tkoeppe deleted the inttypes branch December 14, 2020 13:24


		\xref ISO C~7.20.

		\rSec1[support.start.term]{Start and termination}

[cstdint, cinttypes.syn] Reorganize <cstdint> presentation #843

[cstdint, cinttypes.syn] Reorganize <cstdint> presentation #843

Uh oh!

Conversation

tkoeppe commented Jul 21, 2016 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

tkoeppe commented Nov 18, 2016 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

tkoeppe commented Apr 2, 2018

Uh oh!

jensmaurer commented Sep 15, 2018

Uh oh!

tkoeppe commented Mar 15, 2019

Uh oh!

tkoeppe commented Dec 14, 2020

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

tkoeppe commented Jul 21, 2016 •

edited

Loading

tkoeppe commented Nov 18, 2016 •

edited

Loading