[PATCH RESEND 1/5] input: switch InputOption to use XF86OptionRec storage.

Daniel Stone daniel at fooishbar.org
Thu Oct 20 06:13:33 PDT 2011


Hi,

On Thu, Oct 20, 2011 at 01:05:28PM +1000, Peter Hutterer wrote:
> Use the same struct for both InputOption and XF86OptionRec so we don't need
> to convert to and fro the two in the config backends.
> 
> Signed-off-by: Peter Hutterer <peter.hutterer at who-t.net>
> Reviewed-by: Dan Nicholson <dbn.lists at gmail.com>



> @@ -670,8 +671,8 @@ point_on_screen(ScreenPtr pScreen, int x, int y)
>  static void
>  input_option_free(InputOption *o)
>  {
> -    free(o->key);
> -    free(o->value);
> +    free(o->opt_name);
> +    free(o->opt_val);
>      free(o);
>  }

+   free(o->opt_comment);

With that:
Reviewed-by: Daniel Stone <daniel at fooishbar.org>

Cheers,
Daniel


More information about the xorg-devel mailing list