Michael Thayer <[email protected]> writes: > On 14/04/14 21:35, Keith Packard wrote: >> Julien Cristau <[email protected]> writes: >> >>> Only since >>> http://cgit.freedesktop.org/xorg/xserver/commit/?id=901fbfbbbd71c0d82080957f8ba09eebbc786f2b >>> >>> Which could probably have used a different name to avoid silent >>> breakage. >> >> Yeah, that probably would have been a better change. > Sorry, my fault: it didn't occur to me that this would fail to break > driver builds.
Turns out to be tricky to make this warning into an error because it's
enabled by default, and not with an explicit -W switch. The only way to
get it treated as an error is to turn on -pedantic-errors, but that also
turns on -pedantic, which we don't want as that complains about all of
the GCC extensions that we use. So, you turn that back off with
-Wno-pedantic:
CFLAGS = -pedantic-errors -Wno-pedantic
We're definitely not ready for this though; it also turns casts from
'const char *' to 'char *' into errors. Getting one 'this can never
work' bug converted into a raft of 'this essentially always works'
spurious error messages just isn't helpful.
> Shall I submit another patch to change names? I presume that would
> still be allowed after the end of the release window.
I think so; the alternative is to break hardware cursors on any driver
which hasn't been updated. Anyone else have a second opinion?
--
[email protected]
pgpmXwoSn3zHu.pgp
Description: PGP signature
_______________________________________________ [email protected]: X.Org development Archives: http://lists.x.org/archives/xorg-devel Info: http://lists.x.org/mailman/listinfo/xorg-devel
