Looks good
-Sundar
On Monday 12 August 2013 06:21 PM, Peter Allwin wrote:
Hello!
This patch addresses several Nashorn compatibility issues with in
sa.js and is a merge of my patch [0] and Kris' (kmo) [1] which were
developed separately.
The merged change is identical to [1] with except for line 785 where
I've added a conversion to JavaScript String to ensure the correct
replace method is called.
webrev: http://cr.openjdk.java.net/~allwin/8011888/webrev.01/
<http://cr.openjdk.java.net/%7Eallwin/8011888/webrev.01/>
bug: http://bugs.sun.com/view_bug.do?bug_id=8011888
Thanks!
/Peter
[0] http://cr.openjdk.java.net/~allwin/8011888/webrev.00/
<http://cr.openjdk.java.net/%7Eallwin/8011888/webrev.00/>
[1]
http://mail.openjdk.java.net/pipermail/serviceability-dev/2013-July/010831.html
