RFR 8133564: Runtime - 2nd followup to Validate JVM Command-Line Flag Arguments

[coleen.phillimore at oracle.com]

This looks good apart from fixing the indentation of the arguments for 
all these changes so that they line up.

- CommandLineError::print(verbose,
+ JVMFlag::printError(verbose,
                              "ParallelGCThreads (" UINT32_FORMAT ") or CMSWorkQueueDrainThreshold ("
                              UINTX_FORMAT ") is too large\n",
                              threads, threshold);


Thanks,
Coleen

On 5/23/18 4:25 PM, Gerard Ziemski wrote:
> Hi all,
>
> Please review this follow-up fix that cleans up the following:
>
> #1 We move the error print method out of jvmFlagRangeList and into jvmFlag, which allows us not to include jvmFlagRangeList.hpp just for the print method in various files
>
> #2 We cleanup "utilities/defaultStream.hpp” includes
>
> #3 and #4 in the issue are no longer applicable
>
> Tested locally with various debug/product, headers/no headers, open/closed variations, as well as Mach5 hs-tier1,tier2,tier3 tests, and Mach5 open/closed variations.
>
>
> https://bugs.openjdk.java.net/browse/JDK-8133564
> http://cr.openjdk.java.net/~gziemski/8133564_rev1
>
> cheers