[PATCH] D41458: [libc++][C++17] Elementary string conversions for integral types

[Arthur O'Dwyer via Phabricator via cfe-commits]

Quuxplusone added inline comments.


================
Comment at: include/charconv:89
+_LIBCPP_BEGIN_NAMESPACE_STD
+
+enum class _LIBCPP_ENUM_VIS chars_format
----------------
lichray wrote:
> mclow.lists wrote:
> > lichray wrote:
> > > EricWF wrote:
> > > > We need to hide these names when `_LIBCPP_STD_VER < 17`, since we're not allowed to introduce new names into namespace `std` in older dialects.
> > > But this header is backported to C++11, so I intended to not to guard it.
> > > But this header is backported to C++11, so I intended to not to guard it.
> > 
> > In general, we don't provide new features for old language versions.
> > 
> > The only time we've ever done that is for `string_view`, and I'm **still** not sure I did the right thing there.
> We need to decide on this... From my point of view this header will be widely used by formatting and logging libraries, and it doesn't add much to the community by enforcing C++17 here, especially when the interface we specified are very simple and not using any features beyond C++11.
This question is also relevant to my interests, in re `<memory_resource>`.


Repository:
  rCXX libc++

https://reviews.llvm.org/D41458