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

Sebastian Glita glseba at yahoo.com
Sun Dec 12 10:28:45 PST 2010


Hi,

Many thanks, I have the patch merged in the tree of patches for xorg-server.
S.

----- Original Message ----

From: Simon Thum <simon.thum at gmx.de>
To: Peter Hutterer <peter.hutterer at who-t.net>
Cc: X.Org Devel List <xorg-devel at lists.freedesktop.org>; Sebastian Glita 
<glseba at yahoo.com>
Sent: Sun, December 12, 2010 8:51:00 PM
Subject: Re: [PATCH] xfree86: swap the order to-be-merged lists in 
xf86CollectInputOptions.

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;
>      }

_______________________________________________
xorg-devel at lists.x.org: X.Org development
Archives: http://lists.x.org/archives/xorg-devel
Info: http://lists.x.org/mailman/listinfo/xorg-devel



      


More information about the xorg-devel mailing list