Request for review (S): 7022037: Pause when exiting if a debugger is attached on windows
Staffan Larsen
staffan.larsen at oracle.com
Sat Feb 26 02:13:41 PST 2011
> -----Original Message-----
> From: Keith McGuigan
> Sent: den 25 februari 2011 16:14
> To: Staffan Larsen
> Cc: Dmitriy Samersoff; hotspot-runtime-dev at openjdk.java.net
> Subject: Re: Request for review (S): 7022037: Pause when exiting if a
> debugger is attached on windows
>
>
> I think that looks better, thanks for making those changes. Two
> minor
> issues:
>
> (1) os.hpp:496 - you don't need the "os::" prefix here.
Doh. Copy-paste-error.
> (2) (and this isn't a big deal) Why go through the work in the
> arguments code to determine the launcher and store that result rather
> than just looking at the _sun_java_launcher value at the time that
> you
> need it (in check_and_debugger_and_wait())?
I just wanted to have the code in one place. Arguments::created_by_gamma_launcher() is currently called from three different places.
/Staffan
More information about the hotspot-runtime-dev
mailing list