Skip to content

Commit 11dc505

Browse files
committed
Set priorities based on reflector polls (and age of some old issues in LEWG status)
1 parent 73a6700 commit 11dc505

File tree

6 files changed

+81
-6
lines changed

6 files changed

+81
-6
lines changed

xml/issue2883.xml

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,7 @@
77
<section><sref ref="[string.view]"/></section>
88
<submitter>Switzerland</submitter>
99
<date>3 Feb 2017</date>
10-
<priority>99</priority>
10+
<priority>4</priority>
1111

1212
<discussion>
1313
<b>Addresses CH 9</b>
@@ -32,6 +32,9 @@ it is usable to have <tt>string_view</tt> overloads and there might be many plac
3232

3333
<note>2017-07 Toronto Thurs Issue Prioritization</note>
3434
<p>Status LEWG - they're already looking at this.</p>
35+
36+
<note>2025-10-21; Priority set to 4 based on age of issue and lack of activity.</note>
37+
3538
</discussion>
3639

3740
<resolution>

xml/issue2973.xml

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,7 @@
66
<section><sref ref="[alg.merge]"/></section>
77
<submitter>Billy Robert O'Neal III</submitter>
88
<date>8 Jun 2017</date>
9-
<priority>99</priority>
9+
<priority>4</priority>
1010

1111
<discussion>
1212
<p>
@@ -21,6 +21,9 @@ compares like this, given that users must be prepared for the fallback "not enou
2121

2222
<note>2017-07 Toronto Monday issue prioritization</note>
2323
<p>Status to LEWG</p>
24+
25+
<note>2025-10-21; Priority set to 4 based on age of issue and lack of activity.</note>
26+
2427
</discussion>
2528

2629
<resolution>

xml/issue4259.xml

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

1111
<discussion>
1212
<p>
@@ -64,6 +64,11 @@ else
6464
During reflector discussion of this issue there was a preference to adjust the
6565
proposed wording to use `s.npos` instead of `size_t(-1)`.
6666
</p>
67+
68+
<note>2025-10-21; Reflector poll.</note>
69+
<p>
70+
Set priority to 3 after reflector poll.
71+
</p>
6772
</discussion>
6873

6974
<resolution>

xml/issue4262.xml

Lines changed: 23 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,7 @@
66
<section><sref ref="[alg.copy]"/><sref ref="[alg.remove]"/><sref ref="[alg.unique]"/></section>
77
<submitter>Alex Guteniev</submitter>
88
<date>12 May 2025</date>
9-
<priority>99</priority>
9+
<priority>3</priority>
1010

1111
<discussion>
1212
<p>
@@ -38,6 +38,28 @@ I think there should be a weaker precondition, like <sref ref="[alg.copy]"/>/2:
3838
<i>Preconditions</i>: `result` is not in the range `[first, last)`.
3939
</p>
4040
</blockquote>
41+
42+
<note>2025-10-21; Reflector poll.</note>
43+
<p>
44+
Set priority to 3 after reflector poll.
45+
</p>
46+
<p>
47+
"The proposed resolution is incorrect, because the output iterator could ‘enter’
48+
the range [first, last) via being incremented. If the output iterator is a
49+
reverse iterator, it starts writing to the end of [first,last) before we
50+
need to read from there."
51+
</p>
52+
<p>
53+
"NAD - nothing is broken with the existing constraint. Relaxing it is a design change."
54+
</p>
55+
<p>
56+
"NAD. The right behavior, if the algorithm runs out of output space,
57+
is to return the last processed input iterator and the "next spot"
58+
output iterator. This is what <paper num="P3179"/> (parallel ranges algorithms) does
59+
with its range-as-output algorithms. I would not want the proposed
60+
resolution without this change. Otherwise, we would be introducing
61+
new UB unnecessarily."
62+
</p>
4163
</discussion>
4264

4365
<resolution>

xml/issue4285.xml

Lines changed: 20 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,7 @@
88
</section>
99
<submitter>S. B. Tam</submitter>
1010
<date>27 Jun 2025</date>
11-
<priority>99</priority>
11+
<priority>3</priority>
1212

1313
<discussion>
1414
<p>
@@ -35,6 +35,25 @@ to accept additional formats, ambiguity could arise if the locale's format disag
3535
On POSIX systems, it is possible to query the locale's date format using `nl_langinfo(D_FMT)`.
3636
Maybe an implementation should be allowed to use that format <b>instead</b> of the one indicated by `date_order()`.
3737
</p>
38+
39+
<note>2025-10-21; Reflector poll.</note>
40+
<p>
41+
Set priority to 3 after reflector poll.
42+
</p>
43+
<p>
44+
"The current wording of `do_date_order()` suggests it should return one of the
45+
`dmy`, `mdy`, `ymd`, `ymd`, or `ydm` enumerators as long as the date format
46+
specified by `%x` has no components except day, month, and year.
47+
But that fails to consider alternative separators between the components.
48+
The `kok_IN` locale uses `"%d-%m-%y" which does not contain
49+
"other variable components" but still doesn't match any of the fixed formats
50+
in Table 102.
51+
We need normative wording allowing `no_order`, instead of just a footnoote,
52+
and we need to say `do_get_date` only has to use the formats in the table
53+
when `do_date_order() != no_order`. It should not default to the `mdy` format
54+
when `do_date_order()` returns `no_order`, because if the `mdy` format is
55+
correct then it should have returned `mdy`!"
56+
</p>
3857
</discussion>
3958

4059
<resolution>

xml/issue4378.xml

Lines changed: 24 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,7 @@
88
</section>
99
<submitter>Peter Bindels</submitter>
1010
<date>16 Sep 2025</date>
11-
<priority>99</priority>
11+
<priority>4</priority>
1212

1313
<discussion>
1414
<p>
@@ -48,6 +48,29 @@ This moves it inline with the `data()` specification. Given the hardened precond
4848
not define what the feature does when called with broken preconditions. I have been looking at
4949
the latter but that will be an EWG paper instead.
5050
</p>
51+
52+
<note>2025-10-21; Reflector poll.</note>
53+
<p>
54+
Set priority to 4 after reflector poll.
55+
</p>
56+
<p>
57+
"NAD. `begin() + size()` is not dereferenceable and should remain that way."
58+
</p>
59+
<p>
60+
"Saying "if <code>pos &lt;= size()</code> is redundant given the precondition above."
61+
</p>
62+
<p>
63+
"The resolution removes any guarantee that the value at `str[str.size()]`
64+
is `charT()`. Furthermore, the premise of the issue is incorrect,
65+
returning the address of a different null terminator not belonging to the
66+
string would make traversing it with other string operations UB, so it
67+
has to return a reference to a terminator that's within the same array."
68+
</p>
69+
<p>
70+
"`*(begin() = size())` is UB, but could use `*(data() + size())` instead.
71+
Personally I'd like `*end()` to be valid, but that's certainly LEWG business
72+
requiring a paper."
73+
</p>
5174
</discussion>
5275

5376
<resolution>

0 commit comments

Comments
 (0)