Review request for jdeps option rename to Java-style and enhancements to output dot file format

Alan Bateman Alan.Bateman at oracle.com
Sun Oct 13 20:31:08 UTC 2013


On 11/10/2013 08:08, Mandy Chung wrote:
> There were some discussion of jdeps CLI to follow GNU style option
> or the traditional Java style.  As there will be plans to upgrade
> the JDK tools to GNU style options, it'd be desirable to keep jdeps
> be consistent with other langtools and be upgraded to GNU style in
> a unified manner.
>
> This patch includes the fixes for:
>   JDK-8015912: jdeps output in dot graph format and option to find API 
> dependences
>   JDK-8026255: Switch jdeps to follow traditional Java option style
>
> Webrev at:
> http://cr.openjdk.java.net/~mchung/jdk8/webrevs/8015912/webrev.00/
Command line options are somewhat subjective, I think what you have is 
reasonable. I thought the GNU style that were in the original version 
were okay too (just takes getting used to) but keeping them consistent 
with similarly named options in the other tools is important tool. One 
change that I didn't quite understand is dropping -R for recursive, that 
one seemed intuitive (to me).

Anyway, on the changes then I went through the webrev and I didn't see 
anything obviously wrong. A few comments:

- In PlatformClassPath then alt-rt is special-cased and I think that can 
be removed now.

- Also in PlatformClassPath is special handling of jfxrt.jar (if it is 
present) isn't clear. I recall this came up before and maybe it just 
needs a more detailed comment to explain why it is needed.

- In Analyzer then I think I missed why the results are a LinkedHashMap, 
wouldn't HashMap do?

- A minor comment in JdepsTask but I assume the OutputWriter interface 
can be private

-Alan



More information about the core-libs-dev mailing list