Skip to content

Commit b1a6004

Browse files
committed
Set some priorities based on reflector polls
1 parent fa2af39 commit b1a6004

File tree

7 files changed

+88
-7
lines changed

7 files changed

+88
-7
lines changed

xml/issue4173.xml

Lines changed: 30 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,7 @@
66
<section><sref ref="[containers]"/></section>
77
<submitter>Jonathan Wakely</submitter>
88
<date>21 Nov 2024</date>
9-
<priority>99</priority>
9+
<priority>4</priority>
1010

1111
<discussion>
1212
<p>
@@ -19,6 +19,35 @@ could be confused with node handles for associative containers, and
1919
(based on Wikipedia) has connotations of additional indirection, and something
2020
which would <i>not</i> be invalidated when the underlying storage changes.
2121
</p>
22+
23+
<note>2025-02-07; Reflector poll</note>
24+
<p>
25+
Set priority to 4 after reflector poll.
26+
</p>
27+
<p>
28+
"Maybe 'pointer to elements' or a longer phrase that includes the verb
29+
'invalidates', which has special meaning in this section."
30+
</p>
31+
<p>
32+
"Note that there are cases where we invalidate iterators but not pointers/references."
33+
</p>
34+
<p>
35+
"Maybe define 'addresses' to mean 'pointers and references' since they're
36+
always invalidated at the same time, but iterators are sometimes separate."
37+
</p>
38+
<p>
39+
"Referential element accessors"
40+
</p>
41+
<p>
42+
"Define 'pointer-invalidating' (for both pointers and references) and
43+
'iterator-invalidating', and say that the former always implies the latter.
44+
Maybe also introduce antonyms 'pointer-preserving' and 'iterator-preserving'."
45+
</p>
46+
<p>
47+
"Should be defined in terms of affected elements, e.g.
48+
'pointer-invalidating for any erased elements'."
49+
</p>
50+
2251
</discussion>
2352

2453
<resolution>

xml/issue4176.xml

Lines changed: 7 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,7 @@
66
<section><sref ref="[containers]"/></section>
77
<submitter>Jonathan Wakely</submitter>
88
<date>22 Nov 2024</date>
9-
<priority>99</priority>
9+
<priority>4</priority>
1010

1111
<discussion>
1212
<p>
@@ -24,6 +24,12 @@ in full, and sometimes rely on distant wording that applies to all containers.
2424
It would be easier if `vector::erase` referred back to
2525
<sref ref="[sequence.reqmts]"/>.
2626
</p>
27+
28+
<note>2025-02-07; Reflector poll</note>
29+
<p>
30+
Set priority to 4 after reflector poll.
31+
</p>
32+
2733
</discussion>
2834

2935
<resolution>

xml/issue4180.xml

Lines changed: 8 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,7 @@
66
<section><sref ref="[flat.multiset.modifiers]"/><sref ref="[flat.map.modifiers]"/></section>
77
<submitter>Arthur O'Dwyer</submitter>
88
<date>09 Dec 2024</date>
9-
<priority>99</priority>
9+
<priority>3</priority>
1010

1111
<discussion>
1212
<p>
@@ -34,6 +34,13 @@ While we're touching this paragraph, also resolve the vague word "initializes" t
3434
Editorially, <tt>pair&lt;&hellip;&gt;</tt> is a verbose way to spell the `value_type` of a `flat_map`; we should
3535
be consistent and just say `value_type`.
3636
</p>
37+
38+
<note>2025-02-07; Reflector poll</note>
39+
<p>
40+
Set priority to 3 after reflector poll.
41+
See reflector discussion for comments about proposed resolution.
42+
</p>
43+
3744
</discussion>
3845

3946
<resolution>

xml/issue4181.xml

Lines changed: 14 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,7 @@
66
<section><sref ref="[iterator.range]"/><sref ref="[range.prim.ssize]"/></section>
77
<submitter>Casey Carter</submitter>
88
<date>13 Dec 2024</date>
9-
<priority>99</priority>
9+
<priority>3</priority>
1010

1111
<discussion>
1212
<p>
@@ -18,6 +18,19 @@ Perhaps the `ssize` overloads should return
1818
<tt>static_cast&lt;common_type_t&lt;ptrdiff_t, iter_difference_t&lt;decltype(<i>meow</i>.begin())&gt;&gt;&gt;(<i>meow</i>.size())</tt>
1919
instead when the argument <tt><i>meow</i></tt> models `range`?
2020
</p>
21+
22+
<note>2025-02-07; Reflector poll</note>
23+
<p>
24+
Set priority to 3 after reflector poll.
25+
</p>
26+
<p>
27+
"The behaviour for Ranges regressed due to LWG <iref ref="3403"/>.
28+
It might have been a mistake to broaden the domain of `ranges::ssize` to match
29+
that of `ranges::size`. The latter includes some non-Range arguments to ease
30+
the transition from `std::size`, but thjere was no existing body of pre-C++20
31+
code calling `std::ssize` that needed a transition path."
32+
</p>
33+
2134
</discussion>
2235

2336
<resolution>

xml/issue4190.xml

Lines changed: 11 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,7 @@
66
<section><sref ref="[exec.snd.expos]"/></section>
77
<submitter>Eric Niebler</submitter>
88
<date>2 Jan 2025</date>
9-
<priority>99</priority>
9+
<priority>2</priority>
1010

1111
<discussion>
1212
<p>
@@ -31,6 +31,16 @@ The <tt>sender_in&lt;Sndr, Env&gt;</tt> concept requires that <tt>get_completion
3131
well-formed and that its type is a specialization of <tt>completion_signatures</tt>. But the return type of
3232
`get_completion_signatures(sndr, env)` is exactly the thing this para is trying to define!
3333
</p>
34+
35+
<note>2025-02-07; Reflector poll</note>
36+
<p>
37+
Set priority to 2 after reflector poll.
38+
</p>
39+
<p>
40+
First sentence of p39 should be either "Let the type `Sndr` be ..." or
41+
"Let `Sndr` be ...", but not "Let type `Sndr` be ...".
42+
</p>
43+
3444
</discussion>
3545

3646
<resolution>

xml/issue4196.xml

Lines changed: 11 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,7 @@
66
<section><sref ref="[alg.merge]"/></section>
77
<submitter>Stephen Howe</submitter>
88
<date>22 Jan 2025</date>
9-
<priority>99</priority>
9+
<priority>4</priority>
1010

1111
<discussion>
1212
<p>
@@ -61,6 +61,16 @@ sequence is over, so the 2nd sequence is copied. Only 1 comparison is done, not
6161
<p>
6262
The <a href="https://sgistl.github.io/inplace_merge.html">SGI STL archive</a> correctly says "at most" as well.
6363
</p>
64+
65+
<note>2025-02-07; Reflector poll</note>
66+
<p>
67+
Set priority to 4 after reflector poll.
68+
There were nine votes for P0 (Tentatively Ready), but also several for
69+
NAD Editorial, because <sref ref="[structure.specifications]"/>/7 explicitly
70+
says that all complexity requirements are upper bounds and it's conforming to
71+
do less work. The consensus was that this is still a clarifying improvement.
72+
</p>
73+
6474
</discussion>
6575

6676
<resolution>

xml/issue4202.xml

Lines changed: 7 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,7 @@
66
<section><sref ref="[exec.snd.concepts]"/></section>
77
<submitter>Eric Niebler</submitter>
88
<date>03 Feb 2025</date>
9-
<priority>99</priority>
9+
<priority>1</priority>
1010

1111
<discussion>
1212
<p>
@@ -27,6 +27,12 @@ the `enable_sender` variable template was turned into
2727
an exposition-only <i><code>enable-sender</code></i> concept.
2828
We should roll back that change.
2929
</p>
30+
31+
<note>2025-02-07; Reflector poll</note>
32+
<p>
33+
Set priority to 1 after reflector poll.
34+
</p>
35+
3036
</discussion>
3137

3238
<resolution>

0 commit comments

Comments
 (0)