[PATCH] Xi: change XIChangeDeviceProperty from const pointer to const void *

Chase Douglas chase.douglas at canonical.com
Thu Jan 5 16:48:36 PST 2012


On 01/05/2012 04:15 PM, Peter Hutterer wrote:
> According to Daniel Kurtz, a typedev void *pointer is a atomic type. So a
> const pointer is equivalent to 'void const *' instead of the intended
> 'const void *'.

I believe const pointer is equivalent to 'void * const'. The rule I
remember is that the const keyword affects what is to the left of it.
(Unless there isn't anything to the left, in which case it affects what
is to the right of it, but that's the obvious corollary.) Confusing...

The patch is right, the commit message just needs to be fixed. (And you
have 'typedev' in there too :)

Reviewed-by: Chase Douglas <chase.douglas at canonical.com>


More information about the xorg-devel mailing list