[PATCH synaptics] tools: skip non-existing properties

Chase Douglas chase.douglas at canonical.com
Tue Apr 10 17:26:19 PDT 2012


On 04/10/2012 05:21 PM, Peter Hutterer wrote:
> If a property doesn't exist on a device, skip it.

Does this occur normally? Or does this only happen when mixing and
matching synaptics input modules and synclients?

> Signed-off-by: Peter Hutterer <peter.hutterer at who-t.net>
> ---
>  tools/synclient.c |    9 +++++++++
>  1 files changed, 9 insertions(+), 0 deletions(-)
> 
> diff --git a/tools/synclient.c b/tools/synclient.c
> index 942312a..aef719f 100644
> --- a/tools/synclient.c
> +++ b/tools/synclient.c
> @@ -446,6 +446,13 @@ dp_set_variables(Display *dpy, XDevice* dev, int argc, char *argv[], int first_c
>  	XGetDeviceProperty(dpy, dev, prop, 0, 1000, False, AnyPropertyType,
>  				&type, &format, &nitems, &bytes_after, &data);
>  
> +	if (type == None)
> +	{
> +	    fprintf(stderr, "Property for '%s' not available. Skipping.\n",
> +		    par->name);
> +	    continue;
> +	}
> +
>  	switch(par->prop_format)
>  	{
>  	    case 8:
> @@ -517,6 +524,8 @@ dp_show_settings(Display *dpy, XDevice *dev)
>  	XGetDeviceProperty(dpy, dev, a, 0, len, False,
>  				AnyPropertyType, &type, &format,
>  				&nitems, &bytes_after, &data);
> +	if (type == None)
> +	    continue;
>  
>  	switch(par->prop_format) {
>  	    case 8:

Looks good to me.

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


More information about the xorg-devel mailing list