Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

P1990 Add operator[] to std::initializer_list #737

Closed
wg21bot opened this issue Jan 18, 2020 · 6 comments
Closed

P1990 Add operator[] to std::initializer_list #737

wg21bot opened this issue Jan 18, 2020 · 6 comments
Labels
B2 - improvement Bucket 2 as described by P0592: bug fixes, performance improvements, integration fixes for/between e EWGI Evolution Incubator IS Ship vehicle: IS LEWG Library Evolution ready-for-library-evolution-mailing-list-review This paper needs to be discussed on the Library Evolution mailing list
Milestone

Comments

@wg21bot
Copy link
Collaborator

wg21bot commented Jan 18, 2020

P1990R0 Add operator[] to std::initializer_list (Daniil Goncharov, Antony Polukhin)

@wg21bot wg21bot added LEWG Library Evolution LEWGI Library Evolution Incubator LWG Library labels Jan 18, 2020
@wg21bot wg21bot added this to the 2020-02 milestone Jan 18, 2020
@jensmaurer jensmaurer removed the LWG Library label Jan 18, 2020
@brycelelbach brycelelbach removed the LEWG Library Evolution label Feb 8, 2020
@brycelelbach
Copy link

brycelelbach commented Feb 11, 2020

Prague 2020-02 LEWGI Minutes

P1990R0 std::initializer_list::operator[] Design Review

Chair: Bryce Adelstein Lelbach

Champion: Antony Polukhin

Minute Taker: Ben Craig

Start Review: 2020-02-10 17:04

Add data to this proposal?

POLL: We should promise more committee time to pursuing P1990R0 (std::initializer_list::operator[]), knowing that our time is scarce and this will leave less time for other work.

NO OBJECTION TO UNANIMOUS CONSENT.

Attendance: 19

# of Authors: 1

POLL: Forward P1990R0 (std::initializer_list::operator[]) to LEWG for C++23.

NO OBJECTION TO UNANIMOUS CONSENT.

Attendance: 19

# of Authors: 1

End: 17:18

CONSENSUS: LEWGI sends P1990R0 (std::initializer_list::operator[]), with the guidance below, to LEWG, for C++23.

  • Fix the first example in the comparison table; the left hand side is structurally different from the right (check with Conor Hoekstra for details).
  • Check if these changes will make std::initializer_list satisfy ContiguousView; if so, consider changing the title of the paper to "Make std::initialzier_list satisfy ContiguousView".
  • Sanity check this with someone knowledgable in EWG(I) to make sure this change doesn't have any language implications.

@brycelelbach brycelelbach added LEWG Library Evolution EWGI Evolution Incubator and removed LEWGI Library Evolution Incubator labels Feb 11, 2020
@brycelelbach
Copy link

Botond can you just take a look at this and see if EWGI needs to review it? I'm 85% sure the answer is no.

@brycelelbach brycelelbach added the C++23 Targeted at C++23 label Feb 11, 2020
@brycelelbach
Copy link

Titus, despite my comment to Botond, I believe this is ready for LEWG review.

@brycelelbach brycelelbach added the needs-revision Paper needs changes before it can proceed label Feb 11, 2020
@jensmaurer jensmaurer removed this from the 2020-02 milestone Feb 18, 2020
@jensmaurer jensmaurer removed the C++23 Targeted at C++23 label Mar 7, 2020
@wg21bot
Copy link
Collaborator Author

wg21bot commented Jun 17, 2020

P1990R1 Add operator[] to std::initializer_list (Daniil Goncharov, Antony Polukhin)

@wg21bot wg21bot removed the needs-revision Paper needs changes before it can proceed label Jun 17, 2020
@wg21bot wg21bot added this to the 2020-telecon milestone Jun 17, 2020
@brycelelbach brycelelbach added B2 - improvement Bucket 2 as described by P0592: bug fixes, performance improvements, integration fixes for/between e IS Ship vehicle: IS ready-for-library-evolution-mailing-list-review This paper needs to be discussed on the Library Evolution mailing list labels Aug 25, 2020
@inbal2l
Copy link
Collaborator

inbal2l commented Oct 5, 2020

Feedback from LEWG(I) Mailing list suggests:

  • Note that std::initializer_list is not a general array proxy; it cannot be constructed from an arbitrary (contiguous) range.
    The only way to get one is via a braced-init-list.
  • The difficulty of using an initializer_list as a random access container seems like a feature; it unhelpfully hints at using a std::array, a std::vector, or std::span instead.

Ergo the proposal is currently insufficiently motivated.

Previous discussions:

@inbal2l inbal2l closed this as completed Oct 5, 2020
@inbal2l inbal2l removed the ready-for-library-evolution-mailing-list-review This paper needs to be discussed on the Library Evolution mailing list label Oct 5, 2020
@brycelelbach brycelelbach added the ready-for-library-evolution-mailing-list-review This paper needs to be discussed on the Library Evolution mailing list label Oct 20, 2020
@brycelelbach
Copy link

Re-adding the mailing list tag so this automatically gets triaged if there's a new revision.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
B2 - improvement Bucket 2 as described by P0592: bug fixes, performance improvements, integration fixes for/between e EWGI Evolution Incubator IS Ship vehicle: IS LEWG Library Evolution ready-for-library-evolution-mailing-list-review This paper needs to be discussed on the Library Evolution mailing list
Projects
No open projects
Development

No branches or pull requests

4 participants