RFR: 8286654: Add an optional description accessor on ToolProvider interface

Jonathan Gibbons jjg at openjdk.java.net
Thu May 19 10:23:16 UTC 2022


On Wed, 18 May 2022 14:56:47 GMT, Christian Stein <cstein at openjdk.org> wrote:

> This PR adds an optional description accessor on `ToolProvider` interface.
> 
> This PR also adds short description for each of the listed tool:
> - `jar`
> - `javac`
> - `javadoc`
> - `javap` and `jdeps`
> - `jlink` and `jmod`
> - `jpackage`

Main issue is the one I raised at the beginning, about what grammatical form you should be using for the short description.

Also, is the description a phrase (probably not capitalized, and no terminating period) or a sentence (capitalized with a terminating period.)

src/jdk.compiler/share/classes/com/sun/tools/javac/main/JavacToolProvider.java line 30:

> 28: import com.sun.tools.javac.util.JavacMessages;
> 29: import java.io.PrintWriter;
> 30: import java.util.Optional;

at least in javac, we normally sort `java.*` and `javax.*` imports before other imports.

src/jdk.compiler/share/classes/com/sun/tools/javac/resources/javac.properties line 387:

> 385: 
> 386: javac.description=Read Java declarations and compile them into class files
> 387: 

for your general consideration, what grammatical style do you recommend here? Should it be 3rd person (Reads) instead of 2nd person (Read)

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

PR: https://git.openjdk.java.net/jdk/pull/8772


More information about the core-libs-dev mailing list