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
[container.alloc.reqmts] Better cross-references for allocator-aware … #6189
[container.alloc.reqmts] Better cross-references for allocator-aware … #6189
Conversation
@jwakely, @CaseyCarter Could you please have a look? This looks reasonable to me. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This seems an overall improvement to me.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We need a rebase.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I like the suggestion from Jens, but it's OK either way.
…containers There are now more allocator-aware containers in the standard than when this subclause was first written, so ensure we have call outs to all relevent subclauses. The current wording for 'basic_stacktrace' also shows how containers can properly call out the allocator-aware container requirements, now they have their own titled subclause.
cd2f30b
to
971abf4
Compare
I think I prefer my suggestion over the status quo, even after a night's sleep. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think this now has the requested change from Jens, and is ready to squerge.
…containers
There are now more allocator-aware containers in the standard than when this subclause was first written, so ensure we have call outs to all relevent subclauses.
The current wording for 'basic_stacktrace' also shows how containers can properly call out the allocator-aware container requirements, now they have their own titled subclause.