Request for review (S): 7167069: 6 VM flags crash the VM when queried via jinfo

Stefan Karlsson stefan.karlsson at
Thu May 10 09:37:51 UTC 2012

The fix looks good, but the test needs a copyright header and should 
probably use two spaces for the indentations.


On 2012-05-10 11:07, Bengt Rutisson wrote:
> Hi all,
> Can I have some reviews of this simple fix:
> Sending this to both Runtime and GC. The change is in runtime code, 
> but all the concerned flags are GC flags and I plan to push this 
> through the GC repository unless there are any objections to that.
> When command line flags with the double type were introduced the 
> methods Flag::print_on() and Flag::print_as_flag() were not updated 
> correspondingly. The Flag::print_on() method was updated as part of 
> this change:
> But the Flag::print_as_flag() still didn't handle doubles.
> I created a JTReg test that should test one flag of each supported type.
> Thanks,
> Bengt

More information about the hotspot-gc-dev mailing list