Skip to content

Commit bf14296

Browse files
committed
Set priorities based on reflector polls
1 parent 2204afc commit bf14296

File tree

6 files changed

+84
-9
lines changed

6 files changed

+84
-9
lines changed

xml/issue4109.xml

Lines changed: 26 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,7 @@
66
<section><sref ref="[rand.req.genl]"/></section>
77
<submitter>Peter Dimov</submitter>
88
<date>25 May 2024</date>
9-
<priority>99</priority>
9+
<priority>3</priority>
1010

1111
<discussion>
1212
<p>
@@ -60,6 +60,31 @@ and (1.6) to corresponding <i>extended floating point</i> types and <i>extended
6060
or to the minimum make these extended types conditionally-supported with implementation-defined
6161
semantics (A word of power that is used at several places).
6262
</p>
63+
64+
<note>2025-10-21; Reflector poll.</note>
65+
<p>
66+
Set priority to 3 after reflector poll.
67+
</p>
68+
<p>
69+
"The only benefit of making it UB is allowing it as an extension."
70+
</p>
71+
<p>
72+
"Does anybody actually want to support a BigInt as an extension?
73+
If not, non-integral types should be ill-formed and integral types
74+
not in the list should be conditionally-supported."
75+
</p>
76+
<p>
77+
"I'd like extended arithmetic types to work in general,
78+
it's useful for writing generic math libraries to be able to
79+
generate random numbers of any arithmetic type that the implementation
80+
provides, including short integers and all floating-point types."
81+
</p>
82+
<p>
83+
"Can have overflows here, differences between whether `char` is signed or
84+
unsigned lead to subtle bugs. Most users are better off picking `int` or
85+
`unsigned int` (which is what they get from integral promotions anyway)
86+
and just using values that are in range for their intended destination type."
87+
</p>
6388
</discussion>
6489

6590
<resolution>

xml/issue4221.xml

Lines changed: 10 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,7 @@
88
</section>
99
<submitter>Hewill Kang</submitter>
1010
<date>06 Mar 2025</date>
11-
<priority>99</priority>
11+
<priority>2</priority>
1212

1313
<discussion>
1414
<p>
@@ -50,6 +50,15 @@ int main() {
5050
Although such type might be relatively rare, it does reflect an inconsistency in the general usage of formatting
5151
ranges, which do not support all valid ranges.
5252
</p>
53+
54+
<note>2025-10-21; Reflector poll.</note>
55+
<p>
56+
Set priority to 2 after reflector poll.
57+
</p>
58+
<p>
59+
The majority voted NAD, because the range shown in the issue is nonsensical.
60+
But enough people voted P2 that we should not close it without more discussion.
61+
</p>
5362
</discussion>
5463

5564
<resolution>

xml/issue4273.xml

Lines changed: 17 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,7 @@
88
</section>
99
<submitter>Jiang An</submitter>
1010
<date>03 Jun 2025</date>
11-
<priority>99</priority>
11+
<priority>3</priority>
1212

1313
<discussion>
1414
<p>
@@ -19,6 +19,22 @@ which possibly enables more desired usages.
1919
Currently, libc++ makes these types non-movable, which is arguable undesired per the decision made in
2020
<paper num="P3136R1"/>.
2121
</p>
22+
23+
<note>2025-10-21; Reflector poll.</note>
24+
<p>
25+
Set priority to 3 after reflector poll.
26+
</p>
27+
<p>
28+
"Not seeing the point of this change. For code dealing with generic policies,
29+
it shouldn't be constructing new objects of the policy type (there could be
30+
implementation-defined policies, or ones added in future standards, which
31+
are not default-constructible). For code using a specific policy, just using
32+
the named variable is strictly less typing than constructing the type.
33+
Maybe NAD or LEWG."
34+
"Making them tag types helps e.g. <paper num="P3481"/> where we have to
35+
store execution policies, and in <paper num="P2500"/>."
36+
"Don't you want copy constructible, not default constructible?"
37+
</p>
2238
</discussion>
2339

2440
<resolution>

xml/issue4277.xml

Lines changed: 13 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,7 @@
88
</section>
99
<submitter>Jan Schultke</submitter>
1010
<date>08 Jun 2025</date>
11-
<priority>99</priority>
11+
<priority>4</priority>
1212

1313
<discussion>
1414
<p>
@@ -29,6 +29,18 @@ The <i>Remarks</i> specifications in <sref ref="[alg.min.max]"/> providing exemp
2929
<sref ref="[algorithms.requirements]"/> paragraph 15 suggest that even function templates not
3030
accepting iterators are considered algorithms.
3131
</p>
32+
33+
<note>2025-10-21; Reflector poll.</note>
34+
<p>
35+
Set priority to 4 after reflector poll.
36+
</p>
37+
<p>
38+
"NAD, I don't think people are confused about what an algorithm is."
39+
</p>
40+
<p>
41+
"I'd rather the fix would define 'algorithm', as even the Note below
42+
the change uses that term."
43+
</p>
3244
</discussion>
3345

3446
<resolution>

xml/issue4284.xml

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

4-
<issue num="4284" status="New">
4+
<issue num="4284" status="LEWG">
55
<title>Integer-class types should be formattable</title>
66
<section>
77
<sref ref="[format.formatter.spec]"/>
88
</section>
99
<submitter>Jiang An</submitter>
1010
<date>27 Jun 2025</date>
11-
<priority>99</priority>
11+
<priority>3</priority>
1212

1313
<discussion>
1414
<p>
@@ -66,6 +66,19 @@ This wording is relative to <paper num="N5008"/>.
6666
</superseded>
6767

6868
<note>2025-09-18; Jiang An improves wording</note>
69+
70+
<note>2025-10-21; Reflector poll. Status changed: NEW &rarr; LEWG.</note>
71+
<p>
72+
Set priority to 3 after reflector poll. Send to LEWG.
73+
</p>
74+
<p>
75+
"Which additional operations integer-class types should be required to support
76+
is a design question.
77+
Why `format` but not iostreams? What about `to_chars`/`from_chars`?
78+
All the stuff in <code>&lt;bit&gt;</code>? <code>&lt;utility&gt;</code>?
79+
<code>&lt;cmath&gt;</code>?
80+
You can show examples using any of these and claim 'uncertainty and inconsistency'."
81+
</p>
6982
</discussion>
7083

7184
<resolution>

xml/issue4350.xml

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

4-
<issue num="4350" status="New">
4+
<issue num="4350" status="LEWG">
55
<title>Should flat adaptors use `insert_range` when available?</title>
66
<section>
77
<sref ref="[flat.map]"/>
@@ -29,12 +29,12 @@ with those names must behave as expected, but we already require that for
2929
`std::queue`.
3030
</p>
3131

32-
<note>2025-10-21; Reflector poll.</note>
32+
<note>2025-10-21; Reflector poll. Status changed: New &rarr; LEWG.</note>
3333
<p>
3434
Set priority to 2 after reflector poll.
3535
</p>
3636
<p>
37-
Use <tt>append_range</tt> to be consitient with other adapters.
37+
Use <tt>append_range</tt> to be consistent with other adapters.
3838
Send to LEWG for confirmation.
3939
</p>
4040
</discussion>

0 commit comments

Comments
 (0)