[libcxx-commits] [PATCH] D108502: [libc++][doc] Cleanup, normalize, and update projects status docs

[Kent Ross via Phabricator via libcxx-commits]

mumbleskates added inline comments.


================
Comment at: libcxx/docs/Status/Cxx14.rst:1
-.. _cxx14-status:
-
-================================
-libc++ C++14 Status
-================================
-
-.. include:: ../Helpers/Styles.rst
-
-.. contents::
-   :local:
-
-
-Overview
-================================
-
-In April 2013, the C++ standard committee approved the draft for the next version of the C++ standard, initially known as "C++1y".
-
-The draft standard includes papers and issues that were voted on at the previous three meetings (Kona, Portland, and Bristol).
-
-In August 2014, this draft was approved by ISO as C++14.
-
-This page shows the status of libc++; the status of clang's support of the language features is `here <https://clang.llvm.org/cxx_status.html#cxx14>`__.
-
-The groups that have contributed papers:
-
--  CWG - Core Language Working group
--  LWG - Library working group
--  SG1 - Study group #1 (Concurrency working group)
-
-
-.. _paper-status-cxx14:
-
-Paper Status
-====================================
-
-.. csv-table::
-   :file: Cxx14Papers.csv
-   :header-rows: 1
-   :widths: auto
-
-
-.. _issues-status-cxx14:
-
-Library Working Group Issues Status
-====================================
-
-.. csv-table::
-   :file: Cxx14Issues.csv
-   :header-rows: 1
-   :widths: auto
-
-Last Updated: 25-Mar-2014
+.. _cxx14-status:
+
----------------
Mordante wrote:
> mumbleskates wrote:
> > Mordante wrote:
> > > What has changed in this file?
> > Line endings were CRLF, out of line with other files, for Cxx14-2b and Styles; I can revert all these if that's preferable. Notably this does not affect git blame.
> I don't object against the change, but it was unclear what was changed and it wasn't mentioned in the patch description. I think it would make sense commit this separately, maybe split the patch after review.
It is mentioned in the patch description.

Either way is probably fine. As it is this patch is general docs cleanup, and these are the only 5 files with CRLF in all of libcxx.


================
Comment at: libcxx/docs/Status/FormatPaper.csv:12
+`[format.arg.store] <https://wg21.link/format.arg.store>`_,"Class template format-arg-store",,Mark de Wever,`D103357 <https://llvm.org/D103357>`__,|Review|,
+`[format.formatter.spec] <https://wg21.link/format.formatter.spec>`_,"Formatter specializations - character types",,Mark de Wever,"`D96664 <https://llvm.org/D96664>`__ `D103466 <https://llvm.org/D103466>`__",|Review|,
+[format.formatter.spec],"Formatter specializations - string types",,Mark de Wever,"`D96664 <https://llvm.org/D96664>`__ `D103425 <https://reviews.llvm.org/D103425>`__",|Review|,
----------------
Mordante wrote:
> I like adding the wg21 links, but if we add them we should do it consistently in this document.
I just added it to the first occurrence of every section. I'll add it to every header if that's what you want.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D108502/new/

https://reviews.llvm.org/D108502