[Lldb-commits] [PATCH] debugserver: Disable four-char-constants and format-pedantic warnings

Pavel Labath via lldb-commits lldb-commits at lists.llvm.org
Tue Oct 18 02:13:52 PDT 2016


Hi,

Thanks for the patch. Could you submit the patch through phabricator
<https://reviews.llvm.org> and add Greg Clayton as a reviewer. That
said, my preference would be to actually fix those warnings instead of
silencing them.

pl

On 17 October 2016 at 08:24, 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.
>
> This disables four character literal warnings (so that 'FDSC' in
> debugserver.cpp is allowed) and pedantic formatting warnings, so that
> printf-like functions can pass typed pointers to "%p" in format strings.
>
> 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