Review request for 8011697: ScriptEngine "js" randomly means either "rhino" or "nashorn", but should instead select one

[A. Sundararajan]

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
>