Skip to content

Commit aa824a3

Browse files
committed
Set some priorities after reflector polls
1 parent 74ea453 commit aa824a3

File tree

6 files changed

+85
-7
lines changed

6 files changed

+85
-7
lines changed

xml/issue4068.xml

Lines changed: 16 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -9,14 +9,29 @@
99
</section>
1010
<submitter>Jonathan Wakely</submitter>
1111
<date>19 Apr 2024</date>
12-
<priority>99</priority>
12+
<priority>3</priority>
1313

1414
<discussion>
1515
<p>
1616
During <paper num="P2300"/> review the suggestion was raised that it might
1717
be helpful to be able to talk about "an object that models `Concept`"
1818
instead of "an object whose type models `Concept`" or other more verbose forms.
1919
</p>
20+
21+
<note>2025-10-14; Reflector poll</note>
22+
<p>
23+
Set priority to 3 after reflector poll.
24+
</p>
25+
<p>
26+
"OK with finding a way to shorten this, but not as suggested above.
27+
Overloading the term "models Concept" for both objects and types would be
28+
confusing."
29+
</p>
30+
<p>
31+
"Just say 'a <em>`stoppable-source`</em> object',
32+
or 'a `stoppable_token` object'"
33+
</p>
34+
2035
</discussion>
2136

2237
<resolution>

xml/issue4129.xml

Lines changed: 11 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,7 @@
66
<section><sref ref="[res.on.data.races]"/></section>
77
<submitter>Jiang An</submitter>
88
<date>30 Jul 2024</date>
9-
<priority>99</priority>
9+
<priority>3</priority>
1010

1111
<discussion>
1212
<p>
@@ -20,6 +20,16 @@ accesses some object via a parameter.
2020
It might need to be considered whether we should use the more general term "range" instead of "container",
2121
which will cover <tt>std::span</tt>, etc.
2222
</p>
23+
24+
<note>2025-10-14; Reflector poll</note>
25+
<p>
26+
Set priority to 3 after reflector poll.
27+
</p>
28+
<p>
29+
"'non-const parameters' needs to be fixed, this isn't about
30+
top-level const, e.g. <code>std::string&amp;</code> is not a const type."
31+
</p>
32+
2333
</discussion>
2434

2535
<resolution>

xml/issue4250.xml

Lines changed: 20 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,15 +1,15 @@
11
<?xml version='1.0' encoding='utf-8' standalone='no'?>
22
<!DOCTYPE issue SYSTEM "lwg-issue.dtd">
33

4-
<issue num="4250" status="New">
4+
<issue num="4250" status="LEWG">
55
<title>`swap` overloads for `indirect` and `polymorphic` only found by ADL</title>
66
<section>
77
<sref ref="[indirect.swap]"/>
88
<sref ref="[polymorphic.swap]"/>
99
</section>
1010
<submitter>Jonathan Wakely</submitter>
1111
<date>01 May 2025</date>
12-
<priority>99</priority>
12+
<priority>2</priority>
1313

1414
<discussion>
1515
<p>
@@ -18,6 +18,24 @@ are defined as hidden friends, so are only available via ADL.
1818
This means that calling `std::swap(i1, i2)` will always use the generic
1919
`std::swap` instead of the custom overload for `std::indirect`.
2020
</p>
21+
22+
<note>2025-10-14; Reflector poll</note>
23+
<p>
24+
Set priority to 2 after reflector poll. Status New &rarr; LEWG.
25+
</p>
26+
<p>
27+
This is a broader issue than just `std::indirect` and `std::polymorphic`,
28+
it also affects `std::jthread`, `std::mdspan`, `move_only_function`, `copyable_function`, <em>`node-handle`</em>, `inplace_vector`,
29+
`flat_map` and co., and maybe more.
30+
</p>
31+
<p>
32+
Are we comfortable that defining `swap` as a hidden friend means that
33+
the generic `std::swap` might be suboptimal,
34+
and for allocator-aware types might even do something with different effects?
35+
e.g. if `propagate_on_container_swap` is true but
36+
`propagate_on_container_move_assignment` is false.
37+
</p>
38+
2139
</discussion>
2240

2341
<resolution>

xml/issue4258.xml

Lines changed: 18 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,7 @@
66
<section><sref ref="[allocator.requirements.general]"/></section>
77
<submitter>Jiang An</submitter>
88
<date>5 May 2025</date>
9-
<priority>99</priority>
9+
<priority>4</priority>
1010

1111
<discussion>
1212
<p>
@@ -22,6 +22,23 @@ which is possibly not `size_t`, is required to be usable with the allocator.
2222
Do we want to change these constraints, or change the <i>Cpp17Allocator</i> requirements to
2323
require accepting `size_t` values?
2424
</p>
25+
26+
<note>2025-10-14; Reflector poll</note>
27+
<p>
28+
Set priority to 4 after reflector poll.
29+
</p>
30+
<p>
31+
"The wording technically allows you to write
32+
<code>allocate(same_as&lt;size_type&gt; auto)</code>
33+
and reject any other type."
34+
</p>
35+
<p>
36+
"I imagine lots of code relies on `std::size_t` being accepted.
37+
That's surely a general design issue that's much broader than just the
38+
<em>Cpp17Allocator</em> requirement,
39+
and making the requirement stricter seems evolutionary."
40+
</p>
41+
2542
</discussion>
2643

2744
<resolution>

xml/issue4296.xml

Lines changed: 13 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,7 @@
66
<section><sref ref="[hash.requirements]"/></section>
77
<submitter>Jonathan Wakely</submitter>
88
<date>18 Jul 2025</date>
9-
<priority>99</priority>
9+
<priority>3</priority>
1010

1111
<discussion>
1212
<p>
@@ -44,6 +44,18 @@ a value of type `const H` implies that calling `h(k)` doesn't change `h`.
4444
Maybe we want to clarify it in terms of equality-preserving
4545
<sref ref="[concepts.equality]"/>.
4646
</p>
47+
48+
<note>2025-10-14; Reflector poll</note>
49+
<p>
50+
Set priority to 3 after reflector poll.
51+
</p>
52+
<p>
53+
Could say "Note: Thus<ins>, for a particular value `h`,</ins>
54+
all evaluations of the expression `h(k)` with the same value
55+
for `k` yield the same result for a given execution of the
56+
program."
57+
</p>
58+
4759
</discussion>
4860

4961
<resolution>

xml/issue4388.xml

Lines changed: 7 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,7 @@
66
<section><sref ref="[cstdarg.syn]"/></section>
77
<submitter>Jakub Jelinek</submitter>
88
<date>1 Oct 2025</date>
9-
<priority>99</priority>
9+
<priority>1</priority>
1010

1111
<discussion>
1212
<p>
@@ -28,6 +28,12 @@ It also makes the following undefined:
2828
<pre><code>#define BAD ); format_disk(
2929
va_start(ap, BAD);
3030
</code></pre>
31+
32+
<note>2025-10-14; Reflector poll</note>
33+
<p>
34+
Set priority to 1 after reflector poll.
35+
</p>
36+
3137
</discussion>
3238

3339
<resolution>

0 commit comments

Comments
 (0)