Hi Phil,

bezierCurveTo_error is a constant and should be made static.  Thanks!

-Max

Philip Romanik wrote:
> Hi Max,
> 
> I made a small change to LzDrawView.lzs by moving the error variable 
> from within the bezierCurveTo() function to the LzDrawView class. This 
> will give you the context of the change.
> 
> function bezierCurveTo(cp1x, cp1y, cp2x, cp2y, x, y) {
>     var error = bezierCurveTo_error;
> ...
> }
> var bezierCurveTo_error = 10;
> 
> Is this change acceptable? I didn't find any other references to this 
> variable and I changed it to improve the readability a little. If this 
> is really a constant I can change it to be a static.
> 
> Thanks!
> 
> Phil
> 
> 
>> 2) In LzDrawView, you are changing the API for bezierCurveTo.error.
>> I'd like Max (cc-ed) to say if that is ok or not.
> 

_______________________________________________
Laszlo-dev mailing list
[email protected]
http://www.openlaszlo.org/mailman/listinfo/laszlo-dev

Reply via email to