Details
- Reviewers
• Quuxplusone ldionne - Group Reviewers
Restricted Project - Commits
- rGb9c24257c7b4: Add list of LWG papers accepted by WG21 during the June 2021 plenary
Diff Detail
- Repository
- rG LLVM Github Monorepo
Event Timeline
libcxx/docs/Cxx2bStatusPaperStatus.csv | ||
---|---|---|
19 | Please alphabetize (within lines 14–25, that is); so e.g. P1518R2 should come after P1425R4, and P0401R6 should come first. | |
24 | I believe this was at least partly done in D102119. @cjdb's commit message mentions p2231, although it doesn't explicitly claim to have implemented the whole optional half of the paper. | |
libcxx/docs/FormatIssuePaperStatus.csv | ||
26 | Was p2216 accepted for C++2b in June? If so, then it should also be listed in libcxx/docs/Cxx2bStatusPaperStatus.csv (as well as here: I think this line is good). | |
libcxx/docs/RangesIssuePaperStatus.csv | ||
26–30 | Were these papers accepted for C++2b in June? If so, then they should also be listed in libcxx/docs/Cxx2bStatusPaperStatus.csv (as well as here). |
libcxx/docs/Cxx2bStatusPaperStatus.csv | ||
---|---|---|
24 | I actually would appreciate a follow up PR for that! |
Thanks a lot for updating the documentation!
libcxx/docs/Cxx2bStatusPaperStatus.csv | ||
---|---|---|
26 | Can you add a note that this paper has been retroactively accepted in C++20? (The same for the other papers retroactively accepted to C++20.) | |
libcxx/docs/FormatIssuePaperStatus.csv | ||
26 | It was accepted in the last meeting. Since the paper is backported to C++20 I think it makes sense to place it next to the other format papers accepted in C++20. |
Thanks a lot for updating @cor3ntin!
Ship it once @Quuxplusone is happy, he seems to have taken this one into his hands.
libcxx/docs/Cxx2bStatusPaperStatus.csv | ||
---|---|---|
26 | Actually, based on @Mordante's comments, I'm revising my opinion: If a paper was accepted as a DR against C++20 or earlier (especially, I assume, all the format- and ranges-related papers; but also the constexpr-optional paper p2231 which was DR'ed back to C++20 but not C++17, IIUC), then I see how it makes perfect sense to list them in Cxx2aStatusPaperStatus.csv instead of Cxx2bStatusPaperStatus.csv. I still think that every paper voted in at the June meeting should appear in either Cxx2bStatusPaperStatus.csv or Cxx2aStatusPaperStatus.csv. (That is, no paper should appear solely in RangesIssuePaperStatus.csv. So thank you for fixing that.) It'd be nice if the commit message explained where this data came from. |
Put a link to WG21's github in the commit message
libcxx/docs/Cxx2bStatusPaperStatus.csv | ||
---|---|---|
26 | I think it makes more sense too keep everything under 23 with a note. |
libcxx/docs/Cxx2bStatusPaperStatus.csv | ||
---|---|---|
26 | I would look under C++23 for the status of these papers, so I prefer them in this file. cppreference also lists them under C++23, but with a DR: prefix. |
libcxx/docs/Cxx2bStatusPaperStatus.csv | ||
---|---|---|
26 | What do you think of the current approach? (a note column, see below) |
libcxx/docs/Cxx2bStatusPaperStatus.csv | ||
---|---|---|
26 | I liked the approach when I looked at it before, I just didn't comment on it ;-) |
libcxx/docs/RangesIssuePaperStatus.csv | ||
---|---|---|
26–30 | Indeed, my understanding is that https://wg21.link/P2231R1 was voted to be applied as a DR to C++20 (with the understanding that it's up to implementers to do what they want). |
libcxx/docs/RangesIssuePaperStatus.csv | ||
---|---|---|
26–30 | Yes I still prefer Cxx2b. LWG-issues are backported to previous standards, these are also listed under Cxx2b. |
libcxx/docs/RangesIssuePaperStatus.csv | ||
---|---|---|
26–30 | Well, if @ldionne is leaving it up to me (which is probable IMHO but not 100% clear), I'll make the call for Cxx2a please. |
Please alphabetize (within lines 14–25, that is); so e.g. P1518R2 should come after P1425R4, and P0401R6 should come first.