RFR: 8146632: Add descriptive error messages for removed non-product logging flags.

Coleen Phillimore coleen.phillimore at oracle.com
Tue Mar 15 18:56:09 UTC 2016


Max,

I think this looks fine.  Small nit (for which I don't need to see 
another webrev).

Can you change #endif for the end of the #ifdef PRODUCT to #endif // PRODUCT

This message is so much friendlier.  Oh, and the vm doesn't crash, it 
just exits...

Thanks!
Coleen

On 3/15/16 1:45 PM, Max Ockner wrote:
> Hello again everyone!
>
> Please review this change which adds better error messages for 
> non-product flags that are now converted to Unified Logging.
>
> bug: https://bugs.openjdk.java.net/browse/JDK-8146632
> webrev: http://cr.openjdk.java.net/~mockner/8146632/
>
> Since these options have been removed, we want still want the vm to 
> crash here, but now with an error message giving the correct command 
> line option. The new message looks like this:
>
> > TraceClassInitialization has been removed. Please use 
> -Xlog:classinit instead."
>
> The entire output looks like this:
>
> > TraceClassInitialization has been removed. Please use 
> -Xlog:classinit instead.
> > Error: Could not create the Java Virtual Machine.
> > Error: A fatal exception has occurred. Program will exit.
>
> Tested with jtreg runtime tests. A new test checks for an appropriate 
> error message for every develop flag that has so far been converted to 
> logging.
>
> Thanks,
> Max
>
>



More information about the hotspot-runtime-dev mailing list