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

[allocator.requirements.general] Remove unused provisions #5310

Closed

Conversation

JohelEGP
Copy link
Contributor

No description provided.

@tkoeppe tkoeppe requested a review from jwakely February 22, 2022 12:50
@tkoeppe
Copy link
Contributor

tkoeppe commented Feb 22, 2022

@jensmaurer: I think Table 37 (currently very awkwardly placed after Table 38) should probably be dissolved just like you did for the containers "descriptive variables"!

@jensmaurer
Copy link
Member

Yes, very much so.

Copy link
Member

@jwakely jwakely left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We used to have an unqualified forward but that was replaced in ebd893b for LWG 3315.

@jensmaurer jensmaurer changed the title [allocator.requirements.general] Remove unused provitions [allocator.requirements.general] Remove unused provisions Feb 25, 2022
@jensmaurer
Copy link
Member

@koeppe, do you want to merge this as-is or should I convert it to the non-table form?

@JohelEGP JohelEGP force-pushed the allocator.requirements.general branch from 6b1de5c to 1eca1e4 Compare February 25, 2022 13:19
@tkoeppe
Copy link
Contributor

tkoeppe commented Feb 25, 2022

@jensmaurer If you were able to make the conversion, I'd be very happy to immediately do that instead of this PR!

@tkoeppe
Copy link
Contributor

tkoeppe commented Feb 25, 2022

That is, we can just start with converting the intro, I mean! Not the table itself, that can wait.

@jensmaurer jensmaurer self-assigned this Feb 27, 2022
@JohelEGP JohelEGP deleted the allocator.requirements.general branch April 1, 2023 16:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants