RFR: 8340133: Investigate if the java launcher could give hints about JShell [v3]

Jan Lahoda jlahoda at openjdk.org
Mon Oct 21 10:06:58 UTC 2024


On Wed, 16 Oct 2024 18:01:28 GMT, Roger Riggs <rriggs at 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


More information about the core-libs-dev mailing list