On Sun, Dec 20, 2009 at 10:31 PM, Peter Hutterer <[email protected]>wrote:
> > > - /* FIXME: this will always be the case? */ > > - if (WcmAreaListOverlap(&area, priv->tool->arealist)) > > - return BadValue; > > + /* validate the area */ > > + if (WcmAreaListOverlap(&tmpArea, priv->tool->arealist)) > > + return BadValue; > > + } > > doesn't this do exactly the same as before, except that what was "area" > before is not "tmpArea"? it still compares the new area with the already > existing one and it still always detects an overlap and fails. > > Duh, I didn't test it :(. Trusted my instinct too much. New patch later. Ping
------------------------------------------------------------------------------ This SF.Net email is sponsored by the Verizon Developer Community Take advantage of Verizon's best-in-class app development support A streamlined, 14 day to market process makes app distribution fast and easy Join now and get one step closer to millions of Verizon customers http://p.sf.net/sfu/verizon-dev2dev
_______________________________________________ Linuxwacom-devel mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/linuxwacom-devel
