Skip to content

Commit bb9c61d

Browse files
committed
Update draft Editors' Report with current NB comment dispositions.
1 parent 9164ee0 commit bb9c61d

File tree

1 file changed

+209
-9
lines changed

1 file changed

+209
-9
lines changed

papers/nxxxx.md

Lines changed: 209 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -51,27 +51,228 @@ filed as editorial in the ISO 14882 CD (2019) NB comments,
5151
Except where otherwise noted, these dispositions only represent the current
5252
viewpoint of the Project Editor.
5353

54-
NB 1: Accepted, fixed in abc123
54+
US 021: Accepted, fixed in 50e55ce9.
5555

56-
* Notes
56+
* Split index entries to "block (execution)" and "block (statement)".
57+
* Also added the statement form to Clause 3, Terms and Definitions.
5758

58-
NB 2: No consensus for change.
59+
GB 022: Accepted, fixed in 8cc6bd34.
5960

60-
* Notes
61+
* The relevant change had already been made to [using.headers],
62+
but this corresponding change was missed.
6163

62-
NB 3: Accepted with modifications, fixed by 123abc
64+
JP 023: Accepted, fixed in 868934f7.
6365

64-
* **Modified resolution:** Something.
66+
JP 030: No consensus for change.
6567

66-
NB 4: **XWG to handle issue**
68+
* The text immediately following the grammar makes it clear that
69+
both lowercase `p` and uppercase `P` are permitted.
70+
71+
US 031: No consensus for change.
72+
73+
* The example appears to be valid as-is;
74+
adding `!= 0` does not appear to serve any purpose.
75+
76+
GB 032: Accepted, fixed in 84a1cd53.
77+
78+
US 037: **SG2 to handle issue**
79+
80+
* The proposed change is not editorial.
81+
82+
JP 045: Accepted, fixed in d401794f.
83+
84+
* This fixes a misapplication of the resolution of CWG 2381.
85+
86+
US 047: Accepted with modifications, fixed in 785f689d.
87+
88+
* **Modified resolution:**
89+
Instead of removing the redundant sentence, it was converted into a note
90+
and moved after the following sentence of which it is a consequence.
91+
92+
US 052: **CWG to handle issue**
93+
94+
* The proposed change is not editorial.
95+
96+
JP 057: Accepted, fixed in a06b7a49.
97+
98+
GB 078: Accepted, fixed in e3bb2eba.
99+
100+
* Italicized references to *digit*s that intended to refer to the grammar production.
101+
* Also made some nearby editorial improvements:
102+
added cross-references and fixed an adjacent grammar issue ("is" / "are") in [diff.cpp14.library].
103+
104+
GB 079: **Deferred until US 036 is resolved**
105+
106+
US 085: No consensus for change.
107+
108+
* Per the description in [module.import],
109+
translation units are imported, modules are not.
110+
The wording appears to be correct as-is.
111+
112+
US 088: Accepted with modifications, fixed in d382ea4e.
113+
114+
* **Modified resolution:**
115+
Instead of either of the proposed renamings,
116+
renamed [module.global] to [module.global.frag] and
117+
renamed [cpp.glob.frag] to [cpp.global.frag].
118+
119+
GB 089: Accepted, fixed in fa42d5a6.
120+
121+
US 099: Accepted, fixed in 9b0502bf.
122+
123+
US 106: **CWG to handle issue**
124+
125+
US 108: Accepted, fixed in 2f42a930.
126+
127+
US 153: Propose accepting; **LWG to approve direction**
128+
129+
* Can be handled editorially if approved.
130+
131+
US 154: Duplicate of US 153.
132+
133+
US 155: Propose accepting; **LWG to approve direction**
134+
135+
* Can be handled editorially if approved.
136+
137+
JP 177: Accepted, fixed in 8be40ff0.
138+
139+
* Replaces a reference to ISO/IEC/IEEE 60599 with a reference to the intended ISO/IEC/IEEE 60559.
140+
* IEC 60599 is "Mineral oil filled electrical equipment in service --
141+
guidance on the interpretation of dissolved and free gases analysis"
142+
* ISO/IEC/IEEE 60559 is "Information Technology - Microprocessor Systems --
143+
Floating-Point Arithmetic"
144+
145+
GB 200: No consensus for change.
146+
147+
* The example already includes all combinations of `const`/non-`const` LHS and RHS,
148+
as described by paragraph 6.
149+
The suggested combinations `b == d` and `a == c` are both identical to `a == d`.
150+
(Note that only `a` and `c` are used on the LHS,
151+
and only `b` and `d` are used on the RHS.)
152+
153+
US 216: Propose accepting; **LWG to approve direction**
154+
155+
* Can be handled editorially if approved.
156+
157+
JP 218: **LWG to handle issue**
158+
159+
JP 219: **LWG to handle issue**
160+
161+
GB 225: **LWG to handle issue**
162+
163+
US 242: Propose rejecting; **LWG to approve direction**
164+
165+
* Organizationally, it seems more consistent to list `span` near the
166+
sequence containers, just as we list `string_view` near `string.
167+
168+
US 258: Accepted, fixed in f36f871c.
169+
170+
US 280: **LEWG to select a better name or reject**
171+
172+
* Can be handled editorially if an alternative name is approved.
173+
174+
US 295: **TODO**
175+
176+
JP 314: Accepted with modifications, fixed in 136312cf.
177+
178+
* This is not an ISO "Terms and Definitions" Clause,
179+
so the rules for such a Clause do not apply.
180+
* **Modified resolution:**
181+
Renamed subclause from "Terms and Definitions" to "Preamble"
182+
to make it clear that this is not an ISO "Terms and Definitions" Clause.
183+
Also moved [algorithms.parallel] paragraph 1 into this subclause
184+
to avoid a hanging paragraph.
185+
186+
JP 319: Accepted, fixed in 5ac298cc.
187+
188+
US 325: No consensus for change.
189+
190+
* We do not wish to perform this reorganization at this stage,
191+
but will reconsider the organization of the standard library clauses
192+
for a future standard.
193+
194+
US 327: **SG8 / LEWG to handle issue**
195+
196+
* The proposed change is not editorial.
197+
* Duplicate of PL 326
198+
199+
US 328: **SG8 / LEWG to handle issue**
200+
201+
* The proposed change is not editorial.
202+
203+
US 330: No consensus for change.
204+
205+
* The wording to which this comment is objecting
206+
was removed by [P1355R2](http://wg21.link/p1355r2),
207+
which was adopted by 2019-07 LWG Motion 2.
208+
209+
GB 335: **LWG to handle issue**
210+
211+
* The intent of the wording is unclear;
212+
this issue cannot be resolved editorially.
213+
214+
JP 338: Accepted, fixed in 742f1086.
215+
216+
JP 339: Accepted, fixed in 25a08918.
217+
218+
JP 340: Accepted, fixed in f88f6747.
219+
220+
JP 341: Accepted, fixed in d545c37d.
221+
222+
JP 343: Accepted, fixed in 9252441e.
223+
224+
JP 348: Accepted, fixed in 01dea5f5.
225+
226+
* Adding the default template arguments here is likely not the best resolution in the long term.
227+
The explicit default template arguments, if implemented literally, would disallow including
228+
this header and `<iosfwd>` in the same translation unit.
229+
However, adding them here makes this header consistent with the rest of this Clause.
230+
* An LWG issue will be opened to consider
231+
whether we should require more of the iostreams headers to include `<iosfwd>` and
232+
which header inclusions should result in the default arguments being made available.
233+
234+
JP 349: Accepted, fixed in adcf12ea.
235+
236+
* See JP 348.
237+
238+
JP 350: Accepted, fixed in 53b429c9.
239+
240+
* See JP 348.
241+
242+
US 357: Accepted with modifications, fixed in af747d64.
243+
244+
* **Modified resolution:**
245+
A different revised wording was chosen for the notes:
246+
"The specialization `atomic<bool>` uses the primary template."
247+
248+
US 359: **SG1 / LWG to handle issue**
249+
250+
JP 362: Accepted with modifications, fixed in 195d5bab.
251+
252+
* **Modified resolution:**
253+
In addition to adding the missing `[[nodiscard]]`,
254+
also added the missing `static` and `int` from the synopsis.
255+
256+
CA 366: Accepted, fixed in eaf23727.
257+
258+
JP 373: Accepted with modifications, fixed in 41058d90.
259+
260+
* **Modified resolution:** In addition to requested changes,
261+
also added cross-reference to [temp.pre]
262+
for the *requires-clause* grammar production.
263+
264+
JP 374: Accepted, fixed in dbc3d6a5.
265+
266+
JP 376: Accepted, fixed in 8b5c768e.
67267

68268
## Notable editorial changes
69269

70270
...
71271

72272
### Section label changes
73273

74-
...
274+
* [module.global] -> [module.global.frag]
275+
* [cpp.glob.frag] -> [cpp.global.frag]
75276

76277
## Feature test macros
77278

@@ -84,4 +285,3 @@ This list excludes changes
84285
that do not affect the body text or only affect whitespace or typeface. For a
85286
complete list including such changes (or for the actual deltas applied by these
86287
changes), consult the [draft sources on github](https://github.com/cplusplus/draft/compare/n4835...nxxxx).
87-

0 commit comments

Comments
 (0)