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

[associative.reqmts] Terribly formatted table #78

Closed
sdutoit opened this issue Apr 17, 2013 · 14 comments · Fixed by #4460
Closed

[associative.reqmts] Terribly formatted table #78

sdutoit opened this issue Apr 17, 2013 · 14 comments · Fixed by #4460
Assignees
Labels
big An issue causing a large set of changes, scattered across most of the text. formatting lwg Issue must be reviewed by LWG.
Milestone

Comments

@sdutoit
Copy link
Contributor

sdutoit commented Apr 17, 2013

The table "Associative container requirements" has awful formatting.

@sdutoit
Copy link
Contributor Author

sdutoit commented Apr 17, 2013

For example "a_eq.insert(t)" -> "a_-\neq.insert(t)"

@zygoloid
Copy link
Member

Forwarded to LWG.

@tkoeppe
Copy link
Contributor

tkoeppe commented Dec 5, 2015

Are there any requirement tables that don't have awful formatting? :-)

I have an attempt here to re-layout the tables into paragraphs. I'm waiting for @zygoloid's response.

@burblebee burblebee added the lwg Issue must be reviewed by LWG. label Apr 15, 2016
@jensmaurer jensmaurer added the decision-required A decision of the editorial group (or the Project Editor) is required. label Dec 19, 2016
@jensmaurer jensmaurer added this to the C++20 milestone Mar 2, 2017
@zygoloid
Copy link
Member

zygoloid commented Mar 2, 2017

Our preference is to move away from tables for requirements for C++20.

@jensmaurer jensmaurer removed the decision-required A decision of the editorial group (or the Project Editor) is required. label Mar 2, 2017
@tkoeppe
Copy link
Contributor

tkoeppe commented Nov 7, 2017

Here is an example of a different way to present requirements. The example replaces Table 86 in [container.requirements.general]:

image

The code lives in https://github.com/tkoeppe/draft/tree/table_to_paragraph

@jensmaurer
Copy link
Member

Editorial meeting consensus: Present Thomas' bulleted rewrite to LWG for comment.

@mclow
Copy link
Contributor

mclow commented Jan 2, 2018

Sent to -lib reflector for feedback.

@tkoeppe
Copy link
Contributor

tkoeppe commented Jul 2, 2018

I really hope we can make some progress with this for C++20. There's good guidance from @geoffromer on the reflector, @tkoeppe needs to do some work here.

@tkoeppe tkoeppe self-assigned this Jul 2, 2018
@tkoeppe tkoeppe added the big An issue causing a large set of changes, scattered across most of the text. label Jul 2, 2018
@geoffromer
Copy link
Contributor

For reference, the reflector thread is http://lists.isocpp.org/lib/2018/01/5041.php

@cpplearner
Copy link
Contributor

any update?

@zygoloid
Copy link
Member

Not happening for C++20. Retargeting.

@zygoloid zygoloid modified the milestones: C++20, C++23 Mar 12, 2020
@tkoeppe
Copy link
Contributor

tkoeppe commented Feb 18, 2021

Resumed by Jens's effort in #4460.

@tkoeppe
Copy link
Contributor

tkoeppe commented Feb 22, 2022

@jensmaurer: I take it we can close this now? :-)

@jensmaurer
Copy link
Member

Yes. I've also added a link to the pull request.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
big An issue causing a large set of changes, scattered across most of the text. formatting lwg Issue must be reviewed by LWG.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

9 participants