Hi,

> > +   if (!found)
> > +           return !Success;
> > +
> >     if (!common->wcmModel)
> >     {
> >             common->wcmModel = &usbUnknown;  
> 
>                ^^ isn't this statement supposed to catch the models
> that don't have explicit support and deal with them? I think bailing
> out on unknown models is fixing the wrong problem. what struct
> members are uninitialized?

I somewhat suspected that this would be the case, but since there were
no comments to guide me, I really just took a guess and made X not crash
for me. The patch should ultimately go to the bin as it even breaks
functionality.
As should be obvious through my other e-mails now, the source of the
problem I observed was something completely different after all. Also,
Jason already pointed out to me how the thing is supposed to work
(thanks btw).

Regards,
Julian

------------------------------------------------------------------------------
_______________________________________________
Linuxwacom-devel mailing list
Linuxwacom-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/linuxwacom-devel

Reply via email to