[PATCH xserver 1/2] fix percent options parsing

Peter Hutterer peter.hutterer at who-t.net
Mon Feb 7 17:17:07 PST 2011


On Mon, Feb 07, 2011 at 11:25:46PM +0100, Simon Thum wrote:
> This allows to specify e.g. 0.5% without rounding to zero.
> 
> Signed-off-by: Simon Thum <simon.thum at gmx.de>
> ---
> 
>  All other percent-related functions return double already.
> 
>  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 16c27e5..455cafa 100644
> --- a/hw/xfree86/common/xf86Option.c
> +++ b/hw/xfree86/common/xf86Option.c
> @@ -212,7 +212,7 @@ LookupBoolOption(pointer optlist, const char *name,
> int deflt, Bool markUsed)
>      return deflt;
>  }
>  -static int
> +static double
>  LookupPercentOption(pointer optlist, const char *name, double deflt,
> Bool markUsed)
>  {
>      OptionInfoRec o;
> -- 
> 1.7.3.4

Reviewed-by: Peter Hutterer <peter.hutterer at who-t.net>

but this patch is broken. please re-send it

Cheers,
  Peter


More information about the xorg-devel mailing list