On Tue, 20 Aug 2024 10:15:49 GMT, Pavel Rappo <pra...@openjdk.org> wrote:
>> Could I please get a review for this small clenup? I updated the `--help` >> command output to match that of `man javadoc`. >> >> I also noticed that `--help-extended` was renamed to `--help-extra` at some >> point but doc-comment was not changed, so I changed it. >> I could create a separate issue for it if needed, just to keep a record of >> it. >> >> The new `javadoc --help` output is the following >> >> >> javadoc --help >> Usage: >> javadoc [options] [packagenames] [sourcefiles] [@files] >> where options include: >> @<file> Read options and filenames from file >> --add-modules <module>(,<module>)* >> Root modules to resolve in addition to the initial modules, >> or all modules on the module path if <module> is >> ALL-MODULE-PATH. >> -bootclasspath <path> >> Override location of platform class files used for >> non-modular >> releases >> -breakiterator >> Compute first sentence with BreakIterator >> --class-path <path>, -classpath <path>, -cp <path> >> Specify where to find user class files >> -doclet <class> >> Generate output via alternate doclet >> -docletpath <path> >> Specify where to find doclet class files >> --enable-preview >> Enable preview language features. To be used in >> conjunction with >> either -source or --release. >> -encoding <name> >> Source file encoding name >> -exclude <pkglist> >> Specify a list of packages to exclude >> --expand-requires (transitive|all) >> Instructs the tool to expand the set of modules to be >> documented. By default, only the modules given explicitly >> on >> the command line will be documented. A value of >> "transitive" >> will additionally include all "requires transitive" >> dependencies of those modules. A value of "all" will >> include >> all dependencies of those modules. >> -extdirs <dirlist> >> Override location of installed extensions >> --help, -help, -?, -h >> Display command-line options and exit >> --help-extra, -X >> Print a synopsis of nonstandard options and exit >> -J<flag> Pass <flag> directly to the runtime system >> --limit-modules <module>(,<module>)* >> Limit the u... > > src/jdk.javadoc/share/classes/jdk/javadoc/internal/doclets/formats/html/resources/standard.properties > line 481: > >> 479: >> 480: doclet.usage.docfilessubdirs.description=\ >> 481: Enables deep copying of `doc-files` directories. Subdirectories and >> all\n\ > > Here and elsewhere: backticks mean code font in Markdown files you copied > this message from, but they mean nothing in the console. > > In other properties files, we don't use backticks for such purposes. The > closest we have is this in jar.properties: > > main.usage.summary.try=\ > Try `jar --help' for more information. > > I think we can safely strip backticks throughout this PR. @jonathan-gibbons, > thoughts? They should not be removed/deleted. They could be replaced with single quotes `'` but IIRC they sometimes? have to be doubled in properties files. ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/20618#discussion_r1723446977