Hi,

Few comments on this:

1) This is a JDK repo (where javax.script lives) change and so core-libs-dev review is needed 2) Not sure if it affects any word on spec. - need to go through spec + javadoc comments closely
3) You need a test that demonstrates that order is preserved

Hope this helps,
-Sundar

On Thursday 19 September 2013 11:43 AM, Andreas Rieber wrote:
Hi,

the HashSet used for the script engine factories caused the random behaviour.

Bug:
https://bugs.openjdk.java.net/browse/JDK-8011697

Webrev:
http://cr.openjdk.java.net/~arieber/8011697/webrev.00/

Please review and sponsor required...

thanks
Andreas


Reply via email to