Re: [Linuxwacom-devel] [PATCH] Only update bottomX/bottomY if they haven't been user-configured.

2011-04-03 Thread Ping Cheng
Reviewed-by: Ping Cheng 

Ping
On Sun, Apr 3, 2011 at 7:16 PM, Peter Hutterer wrote:

> wcmInitialToolSize() would xorg-configured overwrite BottomX/Y with the max
> values read from the tablet. Only set the two if they are non-zero (0 is
> the
> default value).
>
> Signed-off-by: Peter Hutterer 
> ---
>  src/xf86Wacom.c |6 --
>  1 files changed, 4 insertions(+), 2 deletions(-)
>
> diff --git a/src/xf86Wacom.c b/src/xf86Wacom.c
> index 6545c90..58d5394 100644
> --- a/src/xf86Wacom.c
> +++ b/src/xf86Wacom.c
> @@ -129,8 +129,10 @@ static void wcmInitialToolSize(InputInfoPtr pInfo)
>priv->resolY = common->wcmResolY;
>}
>
> -   priv->bottomX = priv->maxX;
> -   priv->bottomY = priv->maxY;
> +   if (!priv->bottomX)
> +   priv->bottomX = priv->maxX;
> +   if (!priv->bottomY)
> +   priv->bottomY = priv->maxY;
>
>return;
>  }
> --
> 1.7.4.2
>
>
>
> --
> Create and publish websites with WebMatrix
> Use the most popular FREE web apps or write code yourself;
> WebMatrix provides all the features you need to develop and
> publish your website. http://p.sf.net/sfu/ms-webmatrix-sf
> ___
> Linuxwacom-devel mailing list
> Linuxwacom-devel@lists.sourceforge.net
> https://lists.sourceforge.net/lists/listinfo/linuxwacom-devel
>
--
Create and publish websites with WebMatrix
Use the most popular FREE web apps or write code yourself; 
WebMatrix provides all the features you need to develop and 
publish your website. http://p.sf.net/sfu/ms-webmatrix-sf
___
Linuxwacom-devel mailing list
Linuxwacom-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/linuxwacom-devel


[Linuxwacom-devel] [PATCH] Only update bottomX/bottomY if they haven't been user-configured.

2011-04-03 Thread Peter Hutterer
wcmInitialToolSize() would xorg-configured overwrite BottomX/Y with the max
values read from the tablet. Only set the two if they are non-zero (0 is the
default value).

Signed-off-by: Peter Hutterer 
---
 src/xf86Wacom.c |6 --
 1 files changed, 4 insertions(+), 2 deletions(-)

diff --git a/src/xf86Wacom.c b/src/xf86Wacom.c
index 6545c90..58d5394 100644
--- a/src/xf86Wacom.c
+++ b/src/xf86Wacom.c
@@ -129,8 +129,10 @@ static void wcmInitialToolSize(InputInfoPtr pInfo)
priv->resolY = common->wcmResolY;
}
 
-   priv->bottomX = priv->maxX;
-   priv->bottomY = priv->maxY;
+   if (!priv->bottomX)
+   priv->bottomX = priv->maxX;
+   if (!priv->bottomY)
+   priv->bottomY = priv->maxY;
 
return;
 }
-- 
1.7.4.2


--
Create and publish websites with WebMatrix
Use the most popular FREE web apps or write code yourself; 
WebMatrix provides all the features you need to develop and 
publish your website. http://p.sf.net/sfu/ms-webmatrix-sf
___
Linuxwacom-devel mailing list
Linuxwacom-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/linuxwacom-devel