On Thu, Mar 11, 2010 at 08:22:59AM -0600, Chris Bagwell wrote:
> Hmm, I was staring really hard and still missed this the first time.
> Probably explains some misbehaviour.
> 
> You can add me for the whole series:
> 
> Reviewed-by: Chris Bagwell <[email protected]>
 
thank you. I missed it a few times too and only found it by "accident". I'll
squash this patch one into the other one though before merging so that the
history on master will be cleaner.

Cheers,
  Peter

> On Wed, Mar 10, 2010 at 11:13 PM, Peter Hutterer
> <[email protected]> wrote:
> > Check the first byte of each packet for the bits that define the packet
> > length, not just the first byte in the buffer.
> >
> > Signed-off-by: Peter Hutterer <[email protected]>
> > ---
> >  src/wcmISDV4.c |    1 -
> >  1 files changed, 0 insertions(+), 1 deletions(-)
> >
> > diff --git a/src/wcmISDV4.c b/src/wcmISDV4.c
> > index 1b365dc..0c8b0a8 100644
> > --- a/src/wcmISDV4.c
> > +++ b/src/wcmISDV4.c
> > @@ -398,7 +398,6 @@ static int isdv4Parse(LocalDevicePtr local, const 
> > unsigned char* data, int len)
> >
> >        DBG(10, common, "\n");
> >
> > -       data = common->buffer;
> >        /* choose wcmPktLength if it is not an out-prox event */
> >        if (data[0])
> >                common->wcmPktLength = WACOM_PKGLEN_TPCPEN;
> > --
> > 1.6.6.1
> >
> >

------------------------------------------------------------------------------
Download Intel&#174; Parallel Studio Eval
Try the new software tools for yourself. Speed compiling, find bugs
proactively, and fine-tune applications for parallel performance.
See why Intel Parallel Studio got high marks during beta.
http://p.sf.net/sfu/intel-sw-dev
_______________________________________________
Linuxwacom-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/linuxwacom-devel

Reply via email to