On Wed, 2013-12-11 at 12:24 -0800, Keith Packard wrote:
And fix resulting warnings.
Signed-off-by: Keith Packard kei...@keithp.com
Reviewed-by: Ian Romanick ian.d.roman...@intel.com
Just tried to build this and got:
dri.c: In function 'DRILock':
dri.c:2228:46: error: cast to pointer from
From: Adam Jackson a...@nwnk.net
Date: Fri, 13 Dec 2013 15:42:56 -0500
On Wed, 2013-12-11 at 12:24 -0800, Keith Packard wrote:
And fix resulting warnings.
Signed-off-by: Keith Packard kei...@keithp.com
Reviewed-by: Ian Romanick ian.d.roman...@intel.com
Just tried to build this and
Adam Jackson a...@nwnk.net writes:
Hooray for LP64. These need to be (void *) (uintptr_t) foo.
Yeah, I changed all of the handle casts to add (uintptr_t) and pushed an
updated branch.
--
keith.pack...@intel.com
pgpCtIosWHNjA.pgp
Description: PGP signature
Mark Kettenis mark.kette...@xs4all.nl writes:
It makes much more sense to consistently print drm_context_t as %u or
0x%x. It's an unsigned int on both OpenBSD and Linux, so I suspect
that's the case everywhere.
I'd like to keep the warning patches separate from semantic changes like
this; I
Hooray for LP64. These need to be (void *) (uintptr_t) foo.
Actually, I suspect there is a semantic bug here. If the integers are
smaller than pointers, they don't have as much information content as
pointers, so forcibly shoehorning them into pointers with the help of
additional casts is
And fix resulting warnings.
Signed-off-by: Keith Packard kei...@keithp.com
Reviewed-by: Ian Romanick ian.d.roman...@intel.com
---
hw/xfree86/dri/dri.c | 33 +
1 file changed, 21 insertions(+), 12 deletions(-)
diff --git a/hw/xfree86/dri/dri.c