r217880 - clang-format: Restructure and add missing tests.

Daniel Jasper djasper at google.com
Mon Sep 29 01:18:01 PDT 2014


Fixed in r218597.

On Fri, Sep 26, 2014 at 5:54 PM, Renato Golin <renato.golin at linaro.org>
wrote:

> On 16 September 2014 17:22, Daniel Jasper <djasper at google.com> wrote:
> > +  // For backward compatibility:
> > +  CHECK_PARSE("DerivePointerBinding: true", DerivePointerAlignment,
> true);
> > +  CHECK_PARSE("DerivePointerBinding: false", DerivePointerAlignment,
> false);
>
> Hi Daniel,
>
> GCC 4.9.1 gives me this warning for that last line:
>
> src/llvm/tools/clang/unittests/Format/FormatTest.cpp:8387:1132:
> warning: converting ‘false’ to pointer type for argument 1 of ‘char
> testing::internal::IsNullLiteralHelper(testing::internal::Secret*)’
> [-Wconversion-null]
>    CHECK_PARSE("DerivePointerBinding: false", DerivePointerAlignment,
> false);
>
> cheers,
> --renato
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20140929/22bbed82/attachment.html>


More information about the cfe-commits mailing list