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

Justin Bogner via lldb-commits lldb-commits at lists.llvm.org
Mon Oct 17 11:31:32 PDT 2016


Zachary Turner <zturner at google.com> writes:
> looks fine, thanks

Thanks! r284405.

> 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
>>


More information about the lldb-commits mailing list