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.coopersm...@sun.com
         Oracle Solaris Platform Engineering: X Window System

_______________________________________________
xorg-devel@lists.x.org: X.Org development
Archives: http://lists.x.org/archives/xorg-devel
Info: http://lists.x.org/mailman/listinfo/xorg-devel

Reply via email to