warnings fixes with --enable-debug

Keith Packard keithp at keithp.com
Thu May 22 17:24:16 PDT 2014


Eric Anholt <eric at anholt.net> writes:

> Here's a series for fixing almost all the warnings I get with
> --enable-debug.  I had been building X without --enable-debug (and
> thus without asserts) because mesa forces -O0 with --enable-debug, so
> my normal config script wasn't using the flag.
>
> Remaining warnings include the dead bswap64 in glx (need to fix mesa's
> scripts), and a non-string-literal-to-sprintf warning in x86emu.
>
> Note that in a couple of debug printfs I replaced %x with %p, which
> means that the output will actually change.  I find %p output to be
> much more readable, anyway.

There are a couple of review comments on this series, which otherwise
looks like it's ready for merging.

-- 
keith.packard at intel.com
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 810 bytes
Desc: not available
URL: <http://lists.x.org/archives/xorg-devel/attachments/20140522/e3b15ac3/attachment-0001.sig>


More information about the xorg-devel mailing list