RFR: 8340133: Add concise usage message to the java executable [v5]

Jan Lahoda jlahoda at openjdk.org
Fri Oct 25 10:57:07 UTC 2024


On Thu, 24 Oct 2024 10:10:02 GMT, Jaikiran Pai <jpai at openjdk.org> wrote:

>> Jan Lahoda has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Cleaning up the concise message:
>>   - using 2 spaces instead of 4,
>>   - rewording the "for more use --help" part of the message as suggested to avoid the word "launcher".
>
> src/java.base/share/native/libjli/java.c line 546:
> 
>> 544: 
>> 545:     /* If the user specified neither a class name nor a JAR file */
>> 546:     if (printUsageKind != HELP_NONE) {
> 
> Is the state check for `what` and `mode` no longer needed here?

The idea was that `printUsageKind` should replace that - specifically, `HELP_CONCISE` should be equivalent to the original `what == 0 || mode == LM_UNKNOWN`. The code is a bit cryptic, so I hope I managed to achieve that.

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

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


More information about the core-libs-dev mailing list