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

P0401 Providing size feedback in the Allocator interface #401

Closed
wg21bot opened this issue Jun 23, 2019 · 11 comments · Fixed by cplusplus/draft#4673
Closed

P0401 Providing size feedback in the Allocator interface #401

wg21bot opened this issue Jun 23, 2019 · 11 comments · Fixed by cplusplus/draft#4673
Labels
C++23 Targeted at C++23 LWG Library plenary-approved Papers approved for inclusion in their target vehicle by plenary vote. size - small paper size estimate
Milestone

Comments

@wg21bot
Copy link
Collaborator

wg21bot commented Jun 23, 2019

P0401R1 Providing size feedback in the Allocator interface (Chris Kennelly, Jonathan Wakely)

@wg21bot wg21bot added this to the 2019-07 milestone Jun 23, 2019
@wg21bot wg21bot added LEWG Library Evolution LWG Library labels Jun 23, 2019
@tituswinters tituswinters removed the LWG Library label Jul 5, 2019
@tituswinters
Copy link

Design goes to LEWG first, not targeting C++20.

@jensmaurer jensmaurer added this to Wednesday in LEWG in Cologne 2019 Jul 13, 2019
@tituswinters
Copy link

We want a feature like this.
SF F N A SA
2 9 2 0 0

Name choice:
5 allocate_with_size
4 overallocate
2 allocate_with_oversubscribe
6 allocate_oversize
14 allocate_at_least

Prefer a struct (probably named allocate_at_least_result_t) with named elements (vs requirement of structured bindings).
SF F N A SA
2 8 0 2 1

Revise and return.

@tituswinters tituswinters added LEWG Library Evolution needs-revision Paper needs changes before it can proceed and removed LEWG Library Evolution labels Jul 17, 2019
@jensmaurer jensmaurer modified the milestones: 2019-07, 2019-11 Aug 23, 2019
@jensmaurer jensmaurer removed this from the 2019-11 milestone Dec 12, 2019
@wg21bot
Copy link
Collaborator Author

wg21bot commented Jan 18, 2020

P0401R2 Providing size feedback in the Allocator interface (Chris Kennelly, Jonathan Wakely)

@wg21bot wg21bot added this to the 2020-02 milestone Jan 18, 2020
@jensmaurer jensmaurer removed the needs-revision Paper needs changes before it can proceed label Jan 21, 2020
@wg21bot
Copy link
Collaborator Author

wg21bot commented Mar 24, 2020

P0401R3 Providing size feedback in the Allocator interface (Chris Kennelly, Jonathan Wakely)

@wg21bot wg21bot modified the milestones: 2020-02, 2020-telecon Mar 24, 2020
@FabioFracassi
Copy link
Collaborator

POLL: We prefer to return number of bytes.

Strongly For Weakly For Neutral Weakly Against Strongly Against
0 0 6 6 5

This has no consensus.

POLL: Change the template parameter to sized_ptr_t to pointer type (to support fancy ptr).

Unanimous consent

POLL: We want to deduce the object type from the allocator (alloc_least_n).

Strongly For Weakly For Neutral Weakly Against Strongly Against
4 8 1 0 0

This has consensus.

POLL: We like the design, please revise and do initial wording review (with Pablo, and David Stone) send to LWG (tentatively Ready).

Strongly For Weakly For Neutral Weakly Against Strongly Against
6 12 0 0 0

This has consensus.

@FabioFracassi FabioFracassi added LWG Library needs-revision Paper needs changes before it can proceed tentatively-ready-for-plenary Reviewed between meetings; ready for a vote. labels Apr 17, 2020
@jensmaurer jensmaurer removed this from the 2020-telecon milestone Apr 24, 2020
@brycelelbach brycelelbach added tentatively-ready-for-lwg and removed LEWG Library Evolution tentatively-ready-for-plenary Reviewed between meetings; ready for a vote. labels Aug 25, 2020
@ben-craig ben-craig added B2 - improvement Bucket 2 as described by P0592: bug fixes, performance improvements, integration fixes for/between e size - small paper size estimate labels Oct 17, 2020
@ben-craig
Copy link
Collaborator

R2 needed a revision, R3 has been published. Clearing the needs-revision tag.

@ben-craig ben-craig removed the needs-revision Paper needs changes before it can proceed label Oct 19, 2020
@wg21bot
Copy link
Collaborator Author

wg21bot commented Nov 23, 2020

P0401R4 Providing size feedback in the Allocator interface (Chris Kennelly, Jonathan Wakely)

@wg21bot wg21bot added this to the 2020-telecon milestone Nov 23, 2020
@wg21bot
Copy link
Collaborator Author

wg21bot commented Dec 27, 2020

P0401R5 Providing size feedback in the Allocator interface (Chris Kennelly, Jonathan Wakely)

@wg21bot wg21bot modified the milestones: 2020-telecon, 2021-telecon Dec 27, 2020
@JeffGarland JeffGarland added tentatively-ready-for-plenary Reviewed between meetings; ready for a vote. and removed B2 - improvement Bucket 2 as described by P0592: bug fixes, performance improvements, integration fixes for/between e tentatively-ready-for-lwg labels Jan 22, 2021
@JeffGarland
Copy link
Member

LWG completed review of the paper on 2021-01-22 telecon. Some minor feedback to be checked by Tim after update.

https://wiki.edg.com/bin/view/Wg21telecons2021/P0401-20210122

Adopt P0401 Size Feedback in Allocator for C++23?

F | N | A

8 | 0 | 0

@wg21bot
Copy link
Collaborator Author

wg21bot commented Feb 22, 2021

P0401R6 Providing size feedback in the Allocator interface (Chris Kennelly, Jonathan Wakely)

@JeffGarland JeffGarland added the C++23 Targeted at C++23 label Feb 23, 2021
@cor3ntin cor3ntin added plenary-approved Papers approved for inclusion in their target vehicle by plenary vote. and removed tentatively-ready-for-plenary Reviewed between meetings; ready for a vote. labels Jun 7, 2021
@wg21bot
Copy link
Collaborator Author

wg21bot commented Oct 26, 2021

Adopted 2021-06.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C++23 Targeted at C++23 LWG Library plenary-approved Papers approved for inclusion in their target vehicle by plenary vote. size - small paper size estimate
Projects
No open projects
Development

Successfully merging a pull request may close this issue.

8 participants