RFR: JDK-8221582: Rename jvm-args option to java-options
Alexey Semenyuk
alexey.semenyuk at oracle.com
Thu Mar 28 17:50:33 UTC 2019
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