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

P2580 Tuple protocol for C-style arrays T[N] #1240

Closed
wg21bot opened this issue May 24, 2022 · 3 comments
Closed

P2580 Tuple protocol for C-style arrays T[N] #1240

wg21bot opened this issue May 24, 2022 · 3 comments
Labels
B2 - improvement Bucket 2 as described by P0592: bug fixes, performance improvements, integration fixes for/between e C++26 Targeted at C++26 IS Ship vehicle: IS LEWG Library Evolution needs-revision Paper needs changes before it can proceed ready-for-library-evolution-meeting-review This paper needs to be discussed at a Library Evolution meeting size - medium paper size estimate tuple
Milestone

Comments

@wg21bot
Copy link
Collaborator

wg21bot commented May 24, 2022

P2580R0 Tuple protocol for C-style arrays T[N] (Paolo Di Giglio)

@wg21bot wg21bot added LEWG Library Evolution LEWGI Library Evolution Incubator labels May 24, 2022
@wg21bot wg21bot added this to the 2022-telecon milestone May 24, 2022
@brycelelbach brycelelbach added IS Ship vehicle: IS B2 - improvement Bucket 2 as described by P0592: bug fixes, performance improvements, integration fixes for/between e size - medium paper size estimate C++26 Targeted at C++26 ready-for-library-evolution-mailing-list-review This paper needs to be discussed on the Library Evolution mailing list and removed LEWGI Library Evolution Incubator labels Jun 19, 2022
@brycelelbach
Copy link

@inbal2l @cor3ntin don't do a motion to send this to electronic polling immediately; just do an initial review on the mailing list to get feedback. If the feedback is all positive and no revisions are required, email me so we can discuss next steps.

@brycelelbach brycelelbach added ready-for-library-evolution-meeting-review This paper needs to be discussed at a Library Evolution meeting scheduled-for-library-evolution This paper has been scheduled for one of the groups: LEWG, LEWG Incubator, or a Mailing List review and removed ready-for-library-evolution-mailing-list-review This paper needs to be discussed on the Library Evolution mailing list labels Aug 3, 2022
@brycelelbach brycelelbach added ready-for-library-evolution-mailing-list-review This paper needs to be discussed on the Library Evolution mailing list scheduled-for-library-evolution This paper has been scheduled for one of the groups: LEWG, LEWG Incubator, or a Mailing List review and removed ready-for-library-evolution-meeting-review This paper needs to be discussed at a Library Evolution meeting scheduled-for-library-evolution This paper has been scheduled for one of the groups: LEWG, LEWG Incubator, or a Mailing List review labels Sep 22, 2022
@brycelelbach brycelelbach added ready-for-library-evolution-meeting-review This paper needs to be discussed at a Library Evolution meeting and removed ready-for-library-evolution-mailing-list-review This paper needs to be discussed on the Library Evolution mailing list scheduled-for-library-evolution This paper has been scheduled for one of the groups: LEWG, LEWG Incubator, or a Mailing List review labels Oct 25, 2022
@brycelelbach brycelelbach added the scheduled-for-library-evolution This paper has been scheduled for one of the groups: LEWG, LEWG Incubator, or a Mailing List review label Nov 7, 2022
@brycelelbach
Copy link

The author is unavailable for this paper to be reviewed at the 2022-11 Kona meeting.

@brycelelbach brycelelbach added library-evolution-deferred Ready for review, but should not be scheduled and removed scheduled-for-library-evolution This paper has been scheduled for one of the groups: LEWG, LEWG Incubator, or a Mailing List review labels Nov 7, 2022
@brycelelbach brycelelbach removed the library-evolution-deferred Ready for review, but should not be scheduled label Jan 23, 2023
@jensmaurer jensmaurer modified the milestones: 2022-telecon, 2023-02 Jan 25, 2023
@ben-craig
Copy link
Collaborator

ben-craig commented Feb 10, 2023

2023-02-10 08:30 to 10:15 Issaquah Library Evolution Meeting

P2580R0: Tuple protocol for C-style arrays T[N]

2023-02-10 08:30 to 10:15 UTC-8 Issaquah Library Evolution Minutes

Champion: Paolo Di Giglio (R)

Chair: Ben Craig (IP) & Inbal Levi (IP)

Minute Taker: Steve Downey (IP)

Start: 2023-02-10 09:17 UTC-8

Can this proposal be subsumed by P2141?

POLL: We should promise more committee time to pursuing "Tuple protocol for C-style arrays T[N]", knowing that our time is scarce and this will leave less time for other work.

Strongly Favor Weakly Favor Neutral Weakly Against Strongly Against
2 4 11 11 1

Attendance: 24 local, 16 remote

# of Authors: 1

Author Position: SF

Outcome: No consensus to spend further time.

End: 09:42

Summary

Concerns about the paper:

  • The benefit of this paper seems low. There weren't endorsements along the lines of "I have one of these in my code base", but there were statements along the lines of "I've never needed this".
  • This paper could be subsumed by P2141. Counterpoint though, P2141 may not be implementable as is.
  • This might change the meaning of code with usages of tuple_cat on string literals.
  • Null termination of string literals can lead to surprising tuple_cat behavior
  • This might change the behavior of tuple_flatten

If the issues with code changing meaning can be resolved (or demonstrated to not be a change in meaning), and P2141 stalls, then the committee may still want to look at this, though likely as a low priority addition.

Next Steps

We currently aren't interested, but if you have substantial additional motivation, then we can consider it. Get scary examples from Zach Laine, and scary flatten tuple cases.

@brycelelbach brycelelbach added the needs-revision Paper needs changes before it can proceed label Feb 15, 2023
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 C++26 Targeted at C++26 IS Ship vehicle: IS LEWG Library Evolution needs-revision Paper needs changes before it can proceed ready-for-library-evolution-meeting-review This paper needs to be discussed at a Library Evolution meeting size - medium paper size estimate tuple
Projects
Development

No branches or pull requests

4 participants