Re: SA: Various fixes to sa.js to make it work in JDK8

2013-07-31 Thread Krystal Mok
Hi all, I've just learned that there's a bug number that's exactly what I was trying to fix with this patch: Bug 8011888: http://bugs.sun.com/view_bug.do?bug_id=8011888 If there's already a fix to this bug then I'm fine with it. Just like to see the fix in mainline tip so that CLHSDB would work o

Re: SA: Various fixes to sa.js to make it work in JDK8

2013-07-30 Thread Krystal Mok
Hi Sundar, Thank you very much for taking a look at this. It has been quite a while since sa.js was broken, and it's a pain to use CLHSDB without sa.js functioning. Could you please sponsor this patch, and help me find another Reviewer to get this in? By the way, my OpenJDK username is kmo. Than

Re: SA: Various fixes to sa.js to make it work in JDK8

2013-07-30 Thread A. Sundararajan
Hi Kris, All the JavaScript changes look good to me. Thanks for taking care of rhino-vs-nashorn compatibility issues. PS. I used to work in hotspot serviceability agent in the past. Now, I'm with the nashorn team. Thanks -Sundar On Monday 29 July 2013 01:23 PM, Krystal Mok wrote: Hi all,