On Wed, 11 Nov 2020 04:28:02 GMT, Alexander Matveev <almat...@openjdk.org> 
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

Reply via email to