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

2013-10-15 Thread Alan Bateman
On 15/10/2013 04:38, Mandy Chung wrote: Alan, Thanks for the review. Here is the updated webrev per your feedback: http://cr.openjdk.java.net/~mchung/jdk8/webrevs/8015912/webrev.01/ I have also fixed jdk/test/sun/reflect/CallerSensitive/CallerSensitiveFinder.java due to the change of Clas

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

2013-10-14 Thread Mandy Chung
Alan, Thanks for the review. Here is the updated webrev per your feedback: http://cr.openjdk.java.net/~mchung/jdk8/webrevs/8015912/webrev.01/ I have also fixed jdk/test/sun/reflect/CallerSensitive/CallerSensitiveFinder.java due to the change of ClassFileReader.newInstance method. More co

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

2013-10-14 Thread Paul Sandoz
On Oct 13, 2013, at 10:31 PM, Alan Bateman wrote: > 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

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

2013-10-13 Thread Alan Bateman
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 GN

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

2013-10-11 Thread Mandy Chung
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