On Sun, Oct 17, 2010 at 7:38 PM,  <ch...@cnpbagwell.com> wrote:
> From: Chris Bagwell <ch...@cnpbagwell.com>
>
> Changed !type check to type == EV_SYN to make it clear what its doing.
>
> Signed-off-by: Chris Bagwell <ch...@cnpbagwell.com>

Reviewed by: Ping Cheng <pingli...@gmail.com>

Ping

> ---
>  src/wcmUSB.c |   12 +++++++-----
>  1 files changed, 7 insertions(+), 5 deletions(-)
>
> diff --git a/src/wcmUSB.c b/src/wcmUSB.c
> index eb4778e..131627c 100644
> --- a/src/wcmUSB.c
> +++ b/src/wcmUSB.c
> @@ -936,6 +936,13 @@ static void usbParseEvent(InputInfoPtr pInfo,
>                goto skipEvent;
>        }
>
> +       /* ignore sync windows that contain no data */
> +       if (private->wcmEventCnt == 1 &&
> +           private->wcmEvents->type == EV_SYN) {
> +               DBG(6, common, "no real events received\n");
> +               goto skipEvent;
> +       }
> +
>        /* dispatch all queued events */
>        usbParseEvents(pInfo);
>
> @@ -1206,11 +1213,6 @@ static void usbParseEvents(InputInfoPtr pInfo)
>
>        DBG(6, common, "%d events received\n", private->wcmEventCnt);
>
> -       if (private->wcmEventCnt == 1 && !private->wcmEvents->type) {
> -               DBG(6, common, "no real events received\n");
> -               return;
> -       }
> -
>        channel = usbChooseChannel(common);
>
>        /* couldn't decide channel? invalid data */
> --
> 1.7.3.1
>
>
> ------------------------------------------------------------------------------
> Download new Adobe(R) Flash(R) Builder(TM) 4
> The new Adobe(R) Flex(R) 4 and Flash(R) Builder(TM) 4 (formerly
> Flex(R) Builder(TM)) enable the development of rich applications that run
> across multiple browsers and platforms. Download your free trials today!
> http://p.sf.net/sfu/adobe-dev2dev
> _______________________________________________
> Linuxwacom-devel mailing list
> Linuxwacom-devel@lists.sourceforge.net
> https://lists.sourceforge.net/lists/listinfo/linuxwacom-devel
>

------------------------------------------------------------------------------
Download new Adobe(R) Flash(R) Builder(TM) 4
The new Adobe(R) Flex(R) 4 and Flash(R) Builder(TM) 4 (formerly 
Flex(R) Builder(TM)) enable the development of rich applications that run
across multiple browsers and platforms. Download your free trials today!
http://p.sf.net/sfu/adobe-dev2dev
_______________________________________________
Linuxwacom-devel mailing list
Linuxwacom-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/linuxwacom-devel

Reply via email to