RFR: JDK-8221582: Rename jvm-args option to java-options

Kevin Rushforth kevin.rushforth at oracle.com
Thu Mar 28 17:54:02 UTC 2019


That seems like a good cleanup. It could be done in a follow-on bug 
depending on what Andy wants to do.

-- Kevin


On 3/28/2019 10:50 AM, Alexey Semenyuk wrote:
> Andy,
>
> IMHO if you are renaming defines in C++ code, it makes sense to rename 
> variables/functions too:
> JVMArgs -> JavaArgs in 
> http://cr.openjdk.java.net/~herrick/8221582/webrev.02/src/jdk.jpackage/share/native/libapplauncher/Helpers.cpp.sdiff.html
> Package::ReadJVMArgs -> Package::ReadJavaArgs in 
> http://cr.openjdk.java.net/~herrick/8221582/webrev.02/src/jdk.jpackage/share/native/libapplauncher/Package.cpp.sdiff.html
>
> In general it would make sense to rename JVMArgs in JavaArgs:
> ---
> ASEMENYU-LAP+asemenyu at ASEMENYU-LAP 
> /cygdrive/c/ade/work/as/jds/work/10_sandbox/jdk10/open/src/jdk.jpackage
> $ find . -name '*.cpp' -o -name '*.h' | xargs.exe grep JVMArgs
> ./share/native/libapplauncher/Helpers.cpp: 
> Helpers::GetJVMArgsFromConfig(IPropertyContainer* config) {
> ./share/native/libapplauncher/Helpers.cpp: OrderedMap<TString, 
> TString> JVMArgs =
> ./share/native/libapplauncher/Helpers.cpp: 
> Helpers::GetJVMArgsFromConfig(&propertyFile);
> ./share/native/libapplauncher/Helpers.cpp: 
> Container->AppendSection(keys[CONFIG_SECTION_JVMOPTIONS], JVMArgs);
> ./share/native/libapplauncher/Helpers.h: 
> GetJVMArgsFromConfig(IPropertyContainer* config);
> ./share/native/libapplauncher/JavaVirtualMachine.cpp: 
> options.AppendValues(package.GetJVMArgs());
> ./share/native/libapplauncher/Package.cpp:    ReadJVMArgs(config);
> ./share/native/libapplauncher/Package.cpp:void 
> Package::ReadJVMArgs(ISectionalPropertyContainer* Config) {
> ./share/native/libapplauncher/Package.cpp: FBootFields->FJVMArgs);
> ./share/native/libapplauncher/Package.cpp: FBootFields->FJVMArgs);
> ./share/native/libapplauncher/Package.cpp: FBootFields->FJVMArgs);
> ./share/native/libapplauncher/Package.cpp:OrderedMap<TString, TString> 
> Package::GetJVMArgs() {
> ./share/native/libapplauncher/Package.cpp:    return 
> FBootFields->FJVMArgs;
> ./share/native/libapplauncher/Package.h:    OrderedMap<TString, 
> TString> FJVMArgs;
> ./share/native/libapplauncher/Package.h:    void 
> ReadJVMArgs(ISectionalPropertyContainer* Config);
> ./share/native/libapplauncher/Package.h:    OrderedMap<TString, 
> TString> GetJVMArgs();
> ---
>
> - Alexey
>
> On 3/28/2019 7:07 AM, Andy Herrick wrote:
>> RFR: JDK-8221582: Rename jvm-args option to java-options
>>
>> Please review the jpackage fix for bug [1] at [2].
>>
>> This is a fix for the JDK-8200758-branch branch of the open sandbox 
>> repository (jpackage).
>>
>> [1] - https://bugs.openjdk.java.net/browse/JDK-8221582
>>
>> [2] - http://cr.openjdk.java.net/~herrick/8221582/
>>
>> /Andy
>



More information about the core-libs-dev mailing list