[libcxx-commits] [PATCH] D130031: [libc++][doc] Updates the release notes.

[Mark de Wever via Phabricator via libcxx-commits]

Mordante added a comment.

Thanks for the review! I'll give others some time to chime in before addressing the issues.



================
Comment at: libcxx/docs/ReleaseNotes.rst:46
+The C++20 ``ranges`` library has progressed a lot since the last release and is
+almost complete. The library is considered [experimental|ready for general usage].
+
----------------
philnik wrote:
> Mordante wrote:
> > What will the status of ranges become?
> I think the plan is to make ranges stable.
I heard the same plan ;-) but I'm not deeply involved in that part of the code so I don't know the exact status. Even less involved nowadays since we now have three people very active in ranges area :-)


================
Comment at: libcxx/docs/ReleaseNotes.rst:52-75
+- P0627R6 Function to mark unreachable code
+- P1165R1 Make stateful allocator propagation more consistent for ``operator+(basic_string)``
+- P0674R1 Support arrays in ``make_shared`` and ``allocate_shared``
+- P0980R1 Making ``std::string`` constexpr
+- P2216R3 std::format improvements
+- P0174R2 Deprecating Vestigial Library Parts in C++17
+- N4190 Removing ``auto_ptr``, ``random_shuffle()``, And Old ``<functional>`` Stuff
----------------
philnik wrote:
> I'd like to have some visual indicator that the text is the title of the paper. How about `P1234R5 - This is a really fascinating paper`?
I don't mind a visual indicator, but I felt the parenthesis felt odd. I expect most users will care about the title instead of the number. I'll add them in the next iteration.


================
Comment at: libcxx/docs/ReleaseNotes.rst:133
   errors related to missing declarations in namespace ``std``, make sure you have the
   necessary includes.
 
----------------
philnik wrote:
> Mordante wrote:
> > Are these changes reverted too?
> Yes, the transitive includes of public headers are (almost?) a superset of the ones included in LLVM 14.
Thanks! Then I'll remove this line.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D130031