On Wed, 11 Nov 2020 04:28:02 GMT, Alexander Matveev <[email protected]>
wrote:
> This is regression from JDK-8242302 and for some reason removing -psn
> argument code was removed during refactoring. Fixed be adding removing -psn
> argument back. Also, test was added to test this functionality.
src/jdk.jpackage/macosx/native/common/MacSysInfo.cpp line 90:
> 88: }
> 89: result.push_back(argv[i]);
> 90: }
I'd suggest to use C++ here:
Suggestion:
const tstring psnArgPrefix = "-psn_";
for (int i = progNameMode == ExcludeProgramName ? 1 : 0; i < argc; i++) {
const tstring arg = argv[i];
if (!tstrings::startsWith(arg, psnArgPrefix)) {
result.push_back(arg);
}
}
-------------
PR: https://git.openjdk.java.net/jdk/pull/1154