RFR (XL) 8081519 Split globals.hpp to factor out the Flag class
gerard.ziemski at oracle.com
Thu Mar 29 20:37:41 UTC 2018
That’s a good idea. I’ll work on that and come back with webrev2 shortly...
> On Mar 29, 2018, at 2:17 PM, Vladimir Kozlov <vladimir.kozlov at oracle.com> wrote:
> Should we use this opportunity to move all flags related files into separate directory src/hotspot/share/jvmflags ?
> On 3/29/18 12:01 PM, Gerard Ziemski wrote:
>> Hi all,
>> Please review this large and tedious (sorry), but simple fix that accomplishes the following:
>> #1 factor out the command option flag related APIs out of globals.hpp/.cpp into its own dedicated files, i.e. jvmFlag.hpp/.cpp
>> #2 merge Flag (too generic name) and CommandLineFlag classes and rename them as JVMFlag
>> #3 cleanup globals.hpp includes originally added by the JEP-245
>> Note: the renamed file retain their history, but one needs to add “follow” flag, ex. “hg log -f file”
>> Passes Mach5 hs_tier1-tier5, jtreg/runtime/CommandLine/OptionsValidation/TestOptionsWithRanges tests.
More information about the hotspot-dev