clang static analysis of xorg-server
Alan Coopersmith
Alan.Coopersmith at Sun.COM
Mon Mar 22 14:08:15 PDT 2010
Jeremy Huddleston wrote:
> I aree ... some of the "dead store" issues are more stylistic or future proofing.
>
> The real ones we should consider are the logic errors... garbage assignment and null dereference.
>
> Some of the errors seem outright bogus (http://people.freedesktop.org/~jeremyhu/clang/2010-03-22-1/report-K6CxIC.html#EndPath), so if you see any others like that, let me know. I'm building the latest version of clang right now, and if those oddities are still present, I'll report them to the developers.
That might be pointing out that the definition of XkbControlsEnabledMask is
technically undefined on a 32-bit system, since it should be (1UL << 31) not
(1L << 31) - we've gotten that error in various tools before.
--
-Alan Coopersmith- alan.coopersmith at sun.com
Oracle Solaris Platform Engineering: X Window System
More information about the xorg-devel
mailing list