[PATCH] xfree86: swap the order to-be-merged lists in xf86CollectInputOptions.

Simon Thum simon.thum at gmx.de
Sun Dec 12 10:51:00 PST 2010


On 12/10/2010 04:30 AM, Peter Hutterer wrote:
> Current order causes the user-configured option list to be overwritten with
> the default list supplied by the driver. Swap around so we overwrite the
> driver's default values instead.
> 
> This only affected options supplied by the driver such as XkbLayout in the
> case of evdev.
I guess you got it, but I can't test it due to build problems. However,
consider this
Revieved-by: Simon Thum <simon.thum at gmx.de>

Cheers,

Simon


> 
> Reported-by: Sebastian Glita <glseba at yahoo.com>
> Reported-by: Simon Thum <simon.thum at gmx.de>
> Signed-off-by: Peter Hutterer <peter.hutterer at who-t.net>
> ---
>  hw/xfree86/common/xf86Option.c |    2 +-
>  1 files changed, 1 insertions(+), 1 deletions(-)
> 
> diff --git a/hw/xfree86/common/xf86Option.c b/hw/xfree86/common/xf86Option.c
> index d49aa31..16c27e5 100644
> --- a/hw/xfree86/common/xf86Option.c
> +++ b/hw/xfree86/common/xf86Option.c
> @@ -130,7 +130,7 @@ xf86CollectInputOptions(InputInfoPtr pInfo, const char **defaultOpts)
>      if (defaultOpts) {
>  	XF86OptionPtr tmp =xf86optionListCreate(defaultOpts, -1, 0);
>  	if (pInfo->options)
> -	    pInfo->options = xf86optionListMerge(pInfo->options, tmp);
> +	    pInfo->options = xf86optionListMerge(tmp, pInfo->options);
>  	else
>  	    pInfo->options = tmp;
>      }



More information about the xorg-devel mailing list