Norris Boyd wrote:
> ...
> Thanks for your patience while I investigated these issues. Here's
> some detail on them:
> 
> 370279 "Bad property order in for in loop" - best approached by a Map
> factory for ScriptableObject as has been discussed in 383592 - "Switch
> to HashMap for ScriptableObject property storage" which I think is too
> big a change to put in this release

I'm a bit disappointed that you delay it but I can understand your concerns.

> 368019, 369860, 389278 (various regexp issues) - best addressed by
> 390659 "Replace Rhino regexp implementation with java.util.regex to
> improve performance", assuming that the regexp languages recognized by
> Java and JavaScript are compatible (I haven't analyzed this).

there are some differences between JS and Java Regex. I've started to
work on a patch to use Java Regex in Rhino but it's not yet mature (it
already passes many tests).

> 412247: now implemented
> 392593: fixed
> 374918 - String primitive prototype wrongly resolved when used with
> many top scopes. If this has a workaround can we leave things as-is?

we have a workaround meaning that this doesn't hurt us anymore.
Nevertheless this is still a bug ;-(

> So my proposal is that we wrap up 1.7R1 and then look at
> implementation of 383592 and 390659. I'm willing to finish off 383592
> but I'd need help with 390659.
> 
> How does that sound to you on HtmlUnit?

release early, release often (even if we don't always apply it to ourself)

For us it would be better with 370279 fixed but if you prefer to apply
383592 first in 1.7R2 then I find it perfect if you release 1.7R1 today
and 1.7R2 tomorrow ;-)

We will probably use a custom fixed version of Rhino in HtmlUnit to
allow us to be able to take snapshots of Rhino as well as to apply and
use changes before they are accepted in Rhino. Our interest is that this
custom version becomes useless therefore this won't change anything in
our motivation to contribute to Rhino.

Cheers,
Marc.
-- 
Blog: http://mguillem.wordpress.com
_______________________________________________
dev-tech-js-engine-rhino mailing list
[email protected]
https://lists.mozilla.org/listinfo/dev-tech-js-engine-rhino

Reply via email to