[libcxx-commits] [PATCH] D150060: [libc++][format] Removes optional dependency.
Mark de Wever via Phabricator via libcxx-commits
libcxx-commits at lists.llvm.org
Thu May 18 10:26:51 PDT 2023
Mordante abandoned this revision.
Mordante added a comment.
In D150060#4347359 <https://reviews.llvm.org/D150060#4347359>, @EricWF wrote:
> I'm with ldionne that we should keep using optional here. The correctness seems like a very important quality that we'll be loosing some certainty in.
My main concern was when we need a `formatter<optional, charT>`. In D150509 <https://reviews.llvm.org/D150509> I tested an alternative approach as suggested by Louis, which works without creating header cycles. So when somebody writes a paper for such a formatter we don't risk possible ABI breaks.
(MSVC STL and libstdc++ don't use an optional to implement the locale.)
Since the alternative works, this patch will be abandoned.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D150060/new/
https://reviews.llvm.org/D150060
More information about the libcxx-commits
mailing list