7051182: Regression: : jconsole scripting demo fails with UndeclaredThrowableException
Andreas Rieber
rieberandreas at gmail.com
Tue Jun 11 23:39:27 PDT 2013
Hi Sundar,
i have uploaded two webrevs for it/you. The one line fix for 7051182
fixes also 7023754. This could/should be direct back ported to jdk7
where the problem is the same.
http://cr.openjdk.java.net/~arieber/7023754/webrev.00/
For the compatibility changes i created a new bug similar to the
scriptpad one.
9003978 : jconsole-plugin script demo does not work with nashorn
http://cr.openjdk.java.net/~arieber/9003978/webrev.00/
To run 7023754 with nashorn the changes from 9003978 are required.
cheers
Andreas
On 29.05.13 13:53, A. Sundararajan wrote:
> Hi Andreas,
>
> I'll send out a webrev after build/test of your patch.
>
> Thanks for the contribution!
>
> -Sundar
>
> On Wednesday 29 May 2013 05:04 PM, Andreas Rieber wrote:
>> Hi Sundar,
>>
>> it is all in the same repro (jdk demo) but the fix (public class...
>> in ScriptShellPanel.java) would not work without the changes required
>> for Nashorn. So i guess one patch is fine and shorter this time.
>>
>> Andreas
>>
>>
>>
>> On 29.05.13 11:36, A. Sundararajan wrote:
>>> Hi Andreas,
>>>
>>> If changes are across repos (nashorn, jdk/demo) it is better to have
>>> two patches. But, you could zip everything and send as well.
>>>
>>> Thanks
>>> -Sundar
>>>
>>> On Wednesday 29 May 2013 12:01 PM, Andreas Rieber wrote:
>>>> Hi,
>>>>
>>>> i have a one line fix for this bug but i also have a patch to make
>>>> jconsole-plugin demo working with Nashorn. The problems there are
>>>> similar to scriptpad (sync function, array handling and the new
>>>> JSAdapter __call__).
>>>>
>>>> Do you want two separate patches or is one OK?
>>>>
>>>> cheers
>>>> Andreas
>>>>
>>>
>>
>
>
>
More information about the nashorn-dev
mailing list