Thanks for the quick review.
I changed it to "flag value must be an unsigned 64-bit integer".
Paul
Mandy Chung wrote:
Looks okay to me. Minor nit:
attachListener.cpp line 267: "unsigned integer" in the error message
should be "unsigned 64-bit value" or something like that.
Mandy
Paul Hohensee wrote:
6898160: Need serviceability support for new vm argument type
'uint64_t'.
Webrev here
http://cr.openjdk.java.net/~phh/6898160/webrev.01/
A new vm argument type, 'uint64_t', was added as part of 6887571.
The jdk library
code in jdk/share/native/sun/management/Flag.c that fetches all the
vm product flags
throws an exception if it doesn't understand the type of an
argument. The change
for 6887571 didn't include full serviceability support for uint64_t.
The fix is to add serviceability support for uint64_t. In addition,
if a flag has a type
unknown to the jdk library, we now assert in debug builds and ignore
it in product
builds. Thus, no jdk library change is necessary.
The failing test,
jdk/test/com/sun/management/HotSpotDiagnosticMXBean/GetDiagnosticOptions.java,
now passes.
Thanks,
Paul