[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