[libcxx-commits] [PATCH] D127577: [libc++] Fix std::lower_bound with C++20-hostile iterators
Eric Fiselier via Phabricator via libcxx-commits
libcxx-commits at lists.llvm.org
Sun Jun 12 11:15:01 PDT 2022
EricWF added inline comments.
================
Comment at: libcxx/include/__algorithm/generic_algorithm_helper.h:21
+namespace _Namespace {
+enum _Namespace : bool {
+ _Ranges,
----------------
EricWF wrote:
> Or `struct _Namespace { enum : bool { _Std, _Ranges }; };`. Having the symbol have two entirely different meanings has a mean code smell.
That said, I don't really care to see this addressed at the moment. So just ignore me.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D127577/new/
https://reviews.llvm.org/D127577
More information about the libcxx-commits
mailing list