code review for jps fix (6637230)
Dmitry.Samersoff at oracle.com
Sat Feb 12 01:38:53 PST 2011
Looks OK for me.
+ // leading blank not needed here because of the
+ // next append() call
+ errorString = "-- main class information unavailable";
+ output.append(" ");
May be it could be done as (below)
to keep errorString uniform:
errorString = " -- main class information
String s = MonitoredVmUtil.mainClass(vm,
On 2011-02-12 05:31, Daniel D. Daugherty wrote:
> I have a fix for the following jps bug:
> 6637230: 2/3 jps doesn't work for application waiting for direct attach
> Summary: Properly handle exceptions thrown when querying a monitored VM.
> Here is the URL for the webrev:
> Thanks, in advance, for any reviews.
Java Hotspot development team, SPB04
* There will come soft rains ...
More information about the hotspot-runtime-dev