[Lldb-commits] [PATCH] unittests: Specify types in a bunch of unittest EXPECT's

Zachary Turner via lldb-commits lldb-commits at lists.llvm.org
Mon Oct 17 10:50:21 PDT 2016


looks fine, thanks

On Mon, Oct 17, 2016 at 10:37 AM Justin Bogner via lldb-commits <
lldb-commits at lists.llvm.org> wrote:

> Today I checked out lldb to check that r284364 wouldn't cause any
> trouble, but there were a few warnings that fired when I tried to build
> it, breaking my -Werror build.
>
> The EXPECT and ASSERT macros in gtest don't do the usual arithmetic
> conversions, so there are a number of warnings where we compare
> differently sized ints. This fixes the places that warn to be more
> specific. Okay to commit?
>
> _______________________________________________
> lldb-commits mailing list
> lldb-commits at lists.llvm.org
> http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/lldb-commits/attachments/20161017/228d391f/attachment.html>


More information about the lldb-commits mailing list