[libcxx-commits] [PATCH] D115312: [libc++] [ranges] Simplify and fix a bug in ranges::empty.

Arthur O'Dwyer via Phabricator via libcxx-commits libcxx-commits at lists.llvm.org
Tue Dec 7 18:27:18 PST 2021


Quuxplusone created this revision.
Quuxplusone added reviewers: ldionne, libc++, var-const.
Quuxplusone added a project: libc++.
Quuxplusone requested review of this revision.
Herald added a subscriber: libcxx-commits.
Herald added 1 blocking reviewer(s): libc++.

It was missing the cast to `bool` in `bool(__t.empty())`, as shown in the new test case. There may have been other issues too, but if so, they're fixed now. :)


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D115312

Files:
  libcxx/include/__ranges/empty.h
  libcxx/test/std/ranges/range.access/range.prim/empty.pass.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D115312.392615.patch
Type: text/x-patch
Size: 3994 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/libcxx-commits/attachments/20211208/e09e6f99/attachment.bin>


More information about the libcxx-commits mailing list