RFR(s): 8076012 - SA don't support flags of type size_t
Kim Barrett
kim.barrett at oracle.com
Wed Mar 25 21:25:14 UTC 2015
On Mar 25, 2015, at 3:10 PM, Jesper Wilhelmsson <jesper.wilhelmsson at oracle.com> wrote:
>
> Hi,
>
> Please review this fix for the serviceability agent to support flags of type size_t.
>
> Bug: https://bugs.openjdk.java.net/browse/JDK-8076012
> Webrev: http://cr.openjdk.java.net/~jwilhelm/8076012/webrev.00/
>
> This bug is currently blocking the GC repository from being synced with main after the change to make a bunch of flags size_t.
Looks good.
------------------------------------------------------------------------------
agent/src/share/classes/sun/jvm/hotspot/runtime/VM.java
Changing "a" to "an"
179 Assert.that(isIntx(), "not an intx flag!");
190 Assert.that(isUIntx(), "not an uintx flag!");
Nit-picking: I think the first is correct, but not the second.
And no, I don't need another webrev if you change this.
------------------------------------------------------------------------------
More information about the hotspot-dev
mailing list