Hi Rachel,
On 11/12/2015 8:21 AM, Rachel Protacio wrote:
Hello!
Please review another product flag logging change. This one's pretty
small. Converts TraceMonitorInflation to -Xlog:monitorinflation=debug
Conversion looks fine.
internally, with an alias for the old option. I've again patched in
Max's alias table code in the arguments.cpp and .hpp files since my
previous changeset with that code is currently on hold.
I still have an unanswered question from the other thread about how the
order of options is handled. Primarily whether left-to-right processing
prevails?
I also strongly believe we should be deprecating these flags at the same
time we do the conversion as this is a classic case of deprecation if
ever there was one!
Open webrev: http://cr.openjdk.java.net/~rprotacio/8145153/
Bug: https://bugs.openjdk.java.net/browse/JDK-8145153
On the test, if you use the testlibrary you must also @build it.
Also java -version is sufficient to see some inflating/deflating happen,
and your sample class does nothing to force inflation.
Thanks,
David
Testing: passes jck vm tests, JPRT, and RBT quick and noncolo tests
Thank you!
Rachel