[Lldb-commits] [PATCH] D40537: Simplify UUID::IsValid()

Zachary Turner via lldb-commits lldb-commits at lists.llvm.org
Tue Nov 28 10:28:39 PST 2017


On Tue, Nov 28, 2017 at 10:18 AM Stephane Sezer via Phabricator <
reviews at reviews.llvm.org> wrote:

>
>
> The other alternative seems a bit less explicit to me but I don't mind it
> too much. What's the issue with using `std::any_of` exactly?
>
>
In the sense of correctness, nothing is wrong with using std::any_of.  But
if there's more than one way to do it, and one is easier to read /
understand, I just prefer it.  I'm not blocking the CL or anything, it was
just a suggestion in the interest of readability.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/lldb-commits/attachments/20171128/cbb018e3/attachment-0001.html>


More information about the lldb-commits mailing list