[prev in list] [next in list] [prev in thread] [next in thread] 

List:       freedesktop-xorg-devel
Subject:    resending some missed warning fixes
From:       jamey () minilop ! net (Jamey Sharp)
Date:       2009-10-29 0:48:16
Message-ID: 1256777299-16663-1-git-send-email-jamey () minilop ! net
[Download RAW message or body]

These two patches weren't applied or reviewed. I don't know of a reason
they shouldn't be applied though.

- Suppress GCC warnings like "the address of `u1' will always evaluate
  as `true'".
- Don't cast double to int: use default conversions or explicitly round.


Guillem Jover reviewed this patch and gave suggestions, but I'm
resending it without changes. Details in that mail:

- Suppress certain GCC warnings in auto-generated code.


Two other patches that I'd proposed were controversial at best, and I'm
not resubmitting them at the moment:

- Replace miPointerCurrentScreen with miPointerGetScreen.
- Use PRId32/PRIx32 from inttypes.h for printf of CARD32 values.


If the first three get applied then I think all my xserver patches are
accounted for.

Jamey

[prev in list] [next in list] [prev in thread] [next in thread] 

Configure | About | News | Add a list | Sponsored by KoreLogic