[PATCH synaptics] ClickPad is most definitely a bool option.

Chase Douglas chase.douglas at canonical.com
Wed Apr 25 18:22:44 PDT 2012


On 04/25/2012 05:09 PM, Peter Hutterer wrote:
> Signed-off-by: Peter Hutterer <peter.hutterer at who-t.net>
> ---
>  src/synaptics.c |    2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/src/synaptics.c b/src/synaptics.c
> index d2fe960..2a43f95 100644
> --- a/src/synaptics.c
> +++ b/src/synaptics.c
> @@ -702,7 +702,7 @@ static void set_default_parameters(InputInfoPtr pInfo)
>      pars->tap_move = xf86SetIntOption(opts, "MaxTapMove", tapMove);
>      pars->tap_time_2 = xf86SetIntOption(opts, "MaxDoubleTapTime", 180);
>      pars->click_time = xf86SetIntOption(opts, "ClickTime", 100);
> -    pars->clickpad = xf86SetIntOption(opts, "ClickPad", pars->clickpad); /* Probed */
> +    pars->clickpad = xf86SetBoolOption(opts, "ClickPad", pars->clickpad); /* Probed */
>      pars->fast_taps = xf86SetBoolOption(opts, "FastTaps", FALSE);
>      /* middle mouse button emulation on a clickpad? nah, you're joking */
>      middle_button_timeout = pars->clickpad ? 0 : 75;

Hrm... Don't know how I missed that.

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


More information about the xorg-devel mailing list