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.

Sign up for GitHub

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

Jump to bottom

P2613 Add the missing empty to mdspan #1272

Closed
wg21bot opened this issue Jun 24, 2022 · 5 comments
Closed

P2613 Add the missing empty to mdspan #1272

wg21bot opened this issue Jun 24, 2022 · 5 comments
Labels
C++23 Targeted at C++23 IS Ship vehicle: IS LWG Library mdspan plenary-approved Papers approved for inclusion in their target vehicle by plenary vote. tentatively-ready-for-plenary Reviewed between meetings; ready for a vote.
Milestone
2022-telecon

Comments

@wg21bot
Copy link
Collaborator

wg21bot commented Jun 24, 2022

P2613R0 Add the missing empty to mdspan (Yihe Le)
@wg21bot wg21bot added the LEWG Library Evolution label Jun 24, 2022
@wg21bot wg21bot added this to the 2022-telecon milestone Jun 24, 2022
@cor3ntin cor3ntin added C++23 Targeted at C++23 B2 - improvement Bucket 2 as described by P0592: bug fixes, performance improvements, integration fixes for/between e labels Jun 24, 2022
@brycelelbach
Copy link

brycelelbach commented Jun 29, 2022 •
edited

2022-06-28 Library Evolution Telecon

P2613R0: Add the missing empty to mdspan

2022-06-28 Library Evolution Telecon Minutes

Chair: Bryce Adelstein Lelbach

Minute Taker: Ben Craig

Champion: Mark Hoemmen

Start: 2022-06-28 11:10 Eastern

Does this paper have:

  • Field experience?
    • No, but Library wasn't worried about that.
  • Wording?
    • Yes.
  • Feature test macro?
    • Doesn't need one; hopefully it's going in at the same time as mdspan.

POLL: Send P2613R1 (Add the missing empty to mdspan) to Library for C++23 classified as B2 - Improvement, to be confirmed with a Library Evolution electronic poll.

Strongly Favor Weakly Favor Neutral Weakly Against Strongly Against
13 5 1 0 0

Attendance: 21

# of Authors: 0

Author Position: N/A

Outcome: Strong consensus in favor.

End: 11:16

Summary

We briefly discussed this proposal, which points out that mdspan doesn't have an empty member function.
Library Evolution had consensus to add empty to mdspan.

Next Steps

Take an Library Evolution electronic poll to confirm sending P2613R1 (Add the missing empty to mdspan) to Library for C++23 classified as B2 - Improvement, to be confirmed with a Library Evolution electronic poll.__

@brycelelbach brycelelbach added size - tiny paper size estimate, smaller than small IS Ship vehicle: IS ready-for-library-evolution-electronic-poll This paper needs to undergo a Library Evolution electronic poll needs-revision Paper needs changes before it can proceed scheduled-for-library-evolution This paper has been scheduled for one of the groups: LEWG, LEWG Incubator, or a Mailing List review and removed needs-revision Paper needs changes before it can proceed labels Jun 29, 2022
@brycelelbach brycelelbach added LWG Library lwg-pending LWG Chair needs to disposition labels Jul 8, 2022
@JeffGarland
Copy link
Member

LWG reviewed and approved on 2022-07-08
Notes: https://wiki.edg.com/bin/view/Wg21telecons2022/P2613-20220708

poll: Put P2613r1 into C++23?

F A N
14 0 0

@JeffGarland JeffGarland added tentatively-ready-for-plenary Reviewed between meetings; ready for a vote. and removed lwg-pending LWG Chair needs to disposition B2 - improvement Bucket 2 as described by P0592: bug fixes, performance improvements, integration fixes for/between e size - tiny paper size estimate, smaller than small labels Jul 8, 2022
@wg21bot
Copy link
Collaborator Author

wg21bot commented Jul 21, 2022

P2613R1 Add the missing empty to mdspan (Yihe Le)

@brycelelbach
Copy link

2022-07 Library Evolution Electronic Poll Outcomes

Poll 1.14: Send [P2613R1]: Add The Missing empty To mdspan to Library Working Group for C++23, classified as an improvement of an existing feature ([P0592R4] bucket 2 item).

Strongly Favor Weakly Favor Neutral Weakly Against Strongly Against
16 10 1 1 0

Outcome: Consensus in favor.

@brycelelbach brycelelbach removed LEWG Library Evolution ready-for-library-evolution-electronic-poll This paper needs to undergo a Library Evolution electronic poll scheduled-for-library-evolution This paper has been scheduled for one of the groups: LEWG, LEWG Incubator, or a Mailing List review labels Jul 22, 2022
@cor3ntin cor3ntin added the plenary-approved Papers approved for inclusion in their target vehicle by plenary vote. label Jul 25, 2022
@jensmaurer jensmaurer mentioned this issue Jul 29, 2022
Closed
@tkoeppe
Copy link

tkoeppe commented Aug 11, 2022

This has been applied.

@tkoeppe tkoeppe closed this as completed Aug 11, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
C++23 Targeted at C++23 IS Ship vehicle: IS LWG Library mdspan plenary-approved Papers approved for inclusion in their target vehicle by plenary vote. tentatively-ready-for-plenary Reviewed between meetings; ready for a vote.
Projects
None yet
Milestone
2022-telecon
Development

No branches or pull requests
5 participants
@brycelelbach @cor3ntin @JeffGarland @tkoeppe @wg21bot