On Wed, 16 Oct 2024 18:01:28 GMT, Roger Riggs <rri...@openjdk.org> wrote:

>> Jan Lahoda has updated the pull request incrementally with one additional 
>> commit since the last revision:
>> 
>>   Adjusting the concise help as suggested: 'using main class of a JAR 
>> archive' and '<JarFile>.jar'/'<SourceFile>.java'
>
> src/java.base/share/native/libjli/java.c line 1924:
> 
>> 1922:  */
>> 1923: static void
>> 1924: PrintUsage(JNIEnv* env, jboolean doXUsage, jboolean conciseUsage)
> 
> This would be cleaner to replace the booleans with an enum (concise, X, full).
> Since conciseUsage overrides doXusage, then it should preceed Xusage in the 
> argument list.

Thanks. I've attempt to do that here:
https://github.com/openjdk/jdk/pull/21411/commits/1fe578185e5c8f575dabef5c5fb869e93f218c72

-------------

PR Review Comment: https://git.openjdk.org/jdk/pull/21411#discussion_r1808470949

Reply via email to