Skip to content

Commit eb03859

Browse files
committed
Updates to draft Editors' Report for NB comments:
* add status for US 295 and late comment CH 02. * add hyperlinks for 'fixed in' commits. * update discussion of JP 348 in light of [iosfwd.syn]p1.
1 parent 42eddb8 commit eb03859

File tree

1 file changed

+47
-37
lines changed

1 file changed

+47
-37
lines changed

papers/nxxxx.md

Lines changed: 47 additions & 37 deletions
Original file line numberDiff line numberDiff line change
@@ -51,17 +51,17 @@ 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-
US 021: Accepted, fixed in 50e55ce9.
54+
US 021: Accepted, fixed in [50e55ce9](https://github.com/cplusplus/draft/commit/50e55ce9).
5555

5656
* Split index entries to "block (execution)" and "block (statement)".
5757
* Also added the statement form to Clause 3, Terms and Definitions.
5858

59-
GB 022: Accepted with modifications, fixed in 8cc6bd34.
59+
GB 022: Accepted with modifications, fixed in [8cc6bd34](https://github.com/cplusplus/draft/commit/8cc6bd34).
6060

6161
* The relevant change had already been made to [using.headers],
6262
but this corresponding change was missed.
6363

64-
JP 023: Accepted, fixed in 868934f7.
64+
JP 023: Accepted, fixed in [868934f7](https://github.com/cplusplus/draft/commit/868934f7).
6565

6666
JP 030: No consensus for change.
6767

@@ -73,17 +73,17 @@ US 031: No consensus for change.
7373
* The example appears to be valid as-is;
7474
adding `!= 0` does not appear to serve any purpose.
7575

76-
GB 032: Accepted, fixed in 84a1cd53.
76+
GB 032: Accepted, fixed in [84a1cd53](https://github.com/cplusplus/draft/commit/84a1cd53).
7777

7878
US 037: **SG2 to handle issue**
7979

8080
* The proposed change is not editorial.
8181

82-
JP 045: Accepted, fixed in d401794f.
82+
JP 045: Accepted, fixed in [d401794f](https://github.com/cplusplus/draft/commit/d401794f).
8383

8484
* This fixes a misapplication of the resolution of CWG 2381.
8585

86-
US 047: Accepted with modifications, fixed in 785f689d.
86+
US 047: Accepted with modifications, fixed in [785f689d](https://github.com/cplusplus/draft/commit/785f689d).
8787

8888
* **Modified resolution:**
8989
Instead of removing the redundant sentence, it was converted into a note
@@ -93,9 +93,9 @@ US 052: **CWG to handle issue**
9393

9494
* The proposed change is not editorial.
9595

96-
JP 057: Accepted, fixed in a06b7a49.
96+
JP 057: Accepted, fixed in [a06b7a49](https://github.com/cplusplus/draft/commit/a06b7a49).
9797

98-
GB 078: Accepted, fixed in e3bb2eba.
98+
GB 078: Accepted, fixed in [e3bb2eba](https://github.com/cplusplus/draft/commit/e3bb2eba).
9999

100100
* Italicized references to *digit*s that intended to refer to the grammar production.
101101
* Also made some nearby editorial improvements:
@@ -109,20 +109,20 @@ US 085: No consensus for change.
109109
translation units are imported, modules are not.
110110
The wording appears to be correct as-is.
111111

112-
US 088: Accepted with modifications, fixed in d382ea4e.
112+
US 088: Accepted with modifications, fixed in [d382ea4e](https://github.com/cplusplus/draft/commit/d382ea4e).
113113

114114
* **Modified resolution:**
115115
Instead of either of the proposed renamings,
116116
renamed [module.global] to [module.global.frag] and
117117
renamed [cpp.glob.frag] to [cpp.global.frag].
118118

119-
GB 089: Accepted, fixed in fa42d5a6.
119+
GB 089: Accepted, fixed in [fa42d5a6](https://github.com/cplusplus/draft/commit/fa42d5a6).
120120

121-
US 099: Accepted, fixed in 9b0502bf.
121+
US 099: Accepted, fixed in [9b0502bf](https://github.com/cplusplus/draft/commit/9b0502bf).
122122

123123
US 106: **CWG to handle issue**
124124

125-
US 108: Accepted, fixed in 2f42a930.
125+
US 108: Accepted, fixed in [2f42a930](https://github.com/cplusplus/draft/commit/2f42a930).
126126

127127
US 153: Propose accepting; **LWG to approve direction**
128128

@@ -134,7 +134,7 @@ US 155: Propose accepting; **LWG to approve direction**
134134

135135
* Can be handled editorially if approved.
136136

137-
JP 177: Accepted, fixed in 8be40ff0.
137+
JP 177: Accepted, fixed in [8be40ff0](https://github.com/cplusplus/draft/commit/8be40ff0).
138138

139139
* Replaces a reference to ISO/IEC/IEEE 60599 with a reference to the intended ISO/IEC/IEEE 60559.
140140
* IEC 60599 is "Mineral oil-filled electrical equipment in service --
@@ -165,15 +165,20 @@ US 242: Propose rejecting; **LWG to approve direction**
165165
* Organizationally, it seems more consistent to list `span` near the
166166
sequence containers, just as we list `string_view` near `string.
167167

168-
US 258: Accepted, fixed in f36f871c.
168+
US 258: Accepted, fixed in [f36f871c](https://github.com/cplusplus/draft/commit/f36f871c).
169169

170170
US 280: **LEWG to select a better name or reject**
171171

172172
* Can be handled editorially if an alternative name is approved.
173173

174-
US 295: **TODO**
174+
US 295: Accepted with modifications, fixed in [53f0651e](https://github.com/cplusplus/draft/commit/53f0651e).
175175

176-
JP 314: Accepted with modifications, fixed in 136312cf.
176+
* Instead of proposed change, incorporated the leading
177+
"If `ref_is_glvalue` is `true`" into the bullets
178+
and removed the bullet nesting
179+
to clarify the meaning of the "Otherwise"s.
180+
181+
JP 314: Accepted with modifications, fixed in [136312cf](https://github.com/cplusplus/draft/commit/136312cf).
177182

178183
* This is not an ISO "Terms and Definitions" Clause,
179184
so the rules for such a Clause do not apply.
@@ -183,7 +188,7 @@ JP 314: Accepted with modifications, fixed in 136312cf.
183188
Also moved [algorithms.parallel] paragraph 1 into this subclause
184189
to avoid a hanging paragraph.
185190

186-
JP 319: Accepted, fixed in 5ac298cc.
191+
JP 319: Accepted, fixed in [5ac298cc](https://github.com/cplusplus/draft/commit/5ac298cc).
187192

188193
US 325: No consensus for change.
189194

@@ -211,59 +216,64 @@ GB 335: **LWG to handle issue**
211216
* The intent of the wording is unclear;
212217
this issue cannot be resolved editorially.
213218

214-
JP 338: Accepted, fixed in 742f1086.
219+
JP 338: Accepted, fixed in [742f1086](https://github.com/cplusplus/draft/commit/742f1086).
215220

216-
JP 339: Accepted, fixed in 25a08918.
221+
JP 339: Accepted, fixed in [25a08918](https://github.com/cplusplus/draft/commit/25a08918).
217222

218-
JP 340: Accepted, fixed in f88f6747.
223+
JP 340: Accepted, fixed in [f88f6747](https://github.com/cplusplus/draft/commit/f88f6747).
219224

220-
JP 341: Accepted, fixed in d545c37d.
225+
JP 341: Accepted, fixed in [d545c37d](https://github.com/cplusplus/draft/commit/d545c37d).
221226

222-
JP 343: Accepted, fixed in 9252441e.
227+
JP 343: Accepted, fixed in [9252441e](https://github.com/cplusplus/draft/commit/9252441e).
223228

224-
JP 348: Accepted, fixed in 01dea5f5.
229+
JP 348: Accepted, fixed in [01dea5f5](https://github.com/cplusplus/draft/commit/01dea5f5).
225230

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.
231+
* Per [iosfwd.syn]p1, the duplication of default template arguments
232+
between `<iosfwd>` and `<syncstream>`
233+
does not prevent a translation unit including both.
230234
* 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.
235+
whether we should require more of the iostreams headers to include `<iosfwd>`;
236+
currently only `<ios>` and `<iostream>` are guaranteed to provide the forward declarations.
237+
Similarly LWG should consider whether `<istream>` and `<ostream>` should
238+
be guaranteed to include `<ios>`.
233239

234-
JP 349: Accepted, fixed in adcf12ea.
240+
JP 349: Accepted, fixed in [adcf12ea](https://github.com/cplusplus/draft/commit/adcf12ea).
235241

236242
* See JP 348.
237243

238-
JP 350: Accepted, fixed in 53b429c9.
244+
JP 350: Accepted, fixed in [53b429c9](https://github.com/cplusplus/draft/commit/53b429c9).
239245

240246
* See JP 348.
241247

242-
US 357: Accepted with modifications, fixed in af747d64.
248+
US 357: Accepted with modifications, fixed in [af747d64](https://github.com/cplusplus/draft/commit/af747d64).
243249

244250
* **Modified resolution:**
245251
A different revised wording was chosen for the notes:
246252
"The specialization `atomic<bool>` uses the primary template."
247253

248254
US 359: **SG1 / LWG to handle issue**
249255

250-
JP 362: Accepted with modifications, fixed in 195d5bab.
256+
JP 362: Accepted with modifications, fixed in [195d5bab](https://github.com/cplusplus/draft/commit/195d5bab).
251257

252258
* **Modified resolution:**
253259
In addition to adding the missing `[[nodiscard]]`,
254260
also added the missing `static` and `int` from the synopsis.
255261

256-
CA 366: Accepted, fixed in eaf23727.
262+
CA 366: Accepted, fixed in [eaf23727](https://github.com/cplusplus/draft/commit/eaf23727).
257263

258-
JP 373: Accepted with modifications, fixed in 41058d90.
264+
JP 373: Accepted with modifications, fixed in [41058d90](https://github.com/cplusplus/draft/commit/41058d90).
259265

260266
* **Modified resolution:** In addition to requested changes,
261267
also added cross-reference to [temp.pre]
262268
for the *requires-clause* grammar production.
263269

264-
JP 374: Accepted, fixed in dbc3d6a5.
270+
JP 374: Accepted, fixed in [dbc3d6a5](https://github.com/cplusplus/draft/commit/dbc3d6a5).
271+
272+
JP 376: Accepted, fixed in [8b5c768e](https://github.com/cplusplus/draft/commit/8b5c768e).
273+
274+
### Late comments
265275

266-
JP 376: Accepted, fixed in 8b5c768e.
276+
CH 02: Accepted, fixed in [5ee93fd7](https://github.com/cplusplus/draft/commit/5ee93fd7).
267277

268278
## Notable editorial changes
269279

0 commit comments

Comments
 (0)