Request for review: 6979444

[David Holmes]

Ivan Krylov said the following on 08/30/10 23:26:
>   Bug Link: http://bugs.sun.com/view_bug.do?bug_id=6979444
> Webrev: http://cr.openjdk.java.net/~ikrylov/6979444

The actual link in your html went to the wrong webrev :)

http://cr.openjdk.java.net/%7Ekamg/6930553/webrev.01/

but the real webrev seems ok to me.

Thanks,
David

> 
> The following change effects only nonproduct builds. A new boolean flag 
> is introduced: -XX:+PrintFlagsWithComments
> It works pretty much like -XX:+PrintFlagsInitial (or -Xprintflags) but 
> adds a doc string to each flag that is present in every flag definition 
> in the corresponding *global*.hpp file.
> Minor corrections were made to Flag::print_on method to do proper 
> formatting of all 3 flags (more space for flag name and type, printing 
> of double default values).
> 
> Thanks,
> Ivan
>