Skip to content

P3182 R1 Add container pop methods that return the popped value #1843

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

Open
wg21bot opened this issue Apr 17, 2024 · 3 comments
Open

P3182 R1 Add container pop methods that return the popped value #1843

wg21bot opened this issue Apr 17, 2024 · 3 comments
Labels
LEWG Library Evolution ready-for-library-evolution-meeting-review This paper needs to be discussed at a Library Evolution meeting
Milestone

Comments

@wg21bot
Copy link
Collaborator

wg21bot commented Apr 17, 2024

P3182R0 Add pop_value methods to container adaptors (Brian Bi)

@wg21bot wg21bot added the LEWG Library Evolution label Apr 17, 2024
@wg21bot wg21bot added this to the 2024-telecon milestone Apr 17, 2024
@cor3ntin cor3ntin added the ready-for-library-evolution-mailing-list-review This paper needs to be discussed on the Library Evolution mailing list label Apr 23, 2024
@cor3ntin cor3ntin added 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 and removed ready-for-library-evolution-mailing-list-review This paper needs to be discussed on the Library Evolution mailing list labels Jun 16, 2024
@cor3ntin
Copy link
Collaborator

Mailing list review (June 3 - June 16 2024)

  • Brian started work on a new revision https://isocpp.org/files/papers/D3182R1.html
  • Jonathan started work on a similar design a while back https://kayari.org/cxx/stack_pop.html - I would invite the authors to join forces.
  • We think only supporting container adapters is arbitrary and want to see the design extended to all sequence containers that have pop* methods.
    In its current form, it's unclear to me whether we have a strong consensus to spend time on this.
  • Multiple people have expressed that using move_if_noexcept would increase consensus and make inclusion in the standard more motivated
  • However, using move_if_noexcept has its own set of pitfalls. I don't think we are going to resolve that on the mailing list. We need to poll that specific point when presented in a meeting/telecon
  • There is the idea of a user-specialized type trait to indicate whether a move constructor is exception safe, that might be worth exploring.
  • A third option is to only provide the methods on containers with exception-safe elements

Consequently, we will probably schedule this paper for a telecon once the new draft is published

@wg21bot
Copy link
Collaborator Author

wg21bot commented Jul 16, 2024

P3182R1 Add container pop methods that return the popped value (Brian Bi, Add container pop methods that return the popped value)

@wg21bot wg21bot removed the needs-revision Paper needs changes before it can proceed label Jul 16, 2024
@wg21bot wg21bot changed the title P3182 R0 Add pop_value methods to container adaptors P3182 R1 Add container pop methods that return the popped value Jul 16, 2024
@t3nsor
Copy link

t3nsor commented Aug 4, 2024

Is this paper still in the queue to be scheduled for a telecon?

@jensmaurer jensmaurer modified the milestones: 2024-telecon, 2025-telecon Dec 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
LEWG Library Evolution ready-for-library-evolution-meeting-review This paper needs to be discussed at a Library Evolution meeting
Projects
None yet
Development

No branches or pull requests

4 participants