[sldev] Patches

[Strife Onizuka]

Looks like a positive improvement but there may be a problem with
-2147483648, your patch would read it as -(2147483648) and i'm not sure
exactly how it would handle 2147483648. I'm guessing it would make it
-2147483648 and the neg operation would have no effect. In a worst case
scenario it would round 2147483648 to 2147483647 and you would end up with
-2147483647, which is not good.

As to making existing scripts work as they should, i think it is a good
thing.
Did the old jump bug ever get fixed?

On 3/2/07, Callum Lerwick <seg at haxxed.com> wrote:
>
> On Thu, 2007-03-01 at 12:17 -0800, Phoenix wrote:
> > This patch looks correct, but we are worried about breaking existing
> > scripts. Since we have not yet developed conformance tests for lsl,
> > people here are gun-shy about changing the grammar or implementation.
>
> Well, isn't that what First Look is for? :) Perhaps once the rendering
> improvements are merged into mainline, this patch can be put into the
> next First Look, and let people give it a good pounding for a month or
> two.
>
> My experience with this parsing bug is, it makes scripts that SHOULD
> work not compile, forcing a work-around. (Add a space.) Any currently
> working script should continue to work. Any scripts relying on the
> current buggy parsing, is... some pretty awful code. Of course, its
> inevitable that there's probably some out there *somewhere*...
>
> _______________________________________________
> Click here to unsubscribe or manage your list subscription:
> /index.html
>
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://lists.secondlife.com/pipermail/sldev/attachments/20070303/ce77ea3a/attachment.htm