Sean,
Thank you for adding a test. Rather than creating a dependency on rt.jar
it may be best to have the test use the java.util.jar API to create a
small temporary jar file.
The problem with the dependency on rt.jar is that the test will fail if
run against a development build where the images have not been created (
run against an exploded classes directory ).
-Chris.
On 09/11/2012 04:31, Sean Chou wrote:
Hello Chris, Alan, and Xueming,
I added the testcase, please take a look.
webrev: http://cr.openjdk.java.net/~zhouyx/7201156/webrev.01/
On Wed, Nov 7, 2012 at 6:16 PM, Chris Hegarty <chris.hega...@oracle.com
<mailto:chris.hega...@oracle.com>> wrote:
The change looks fine to me.
I wonder if it is worth creating an automatic regression test to
verify this change ( so an future regression in behavior gets caught
early ). You could call sun.tools.jar.Main directly passing suitable
streams to check the output.
-Chris.
On 07/11/2012 08:31, Sean Chou wrote:
Hello,
This is the suggested fix described in sun bug 7201156 page.
Please take a
look.
sunbug:
http://bugs.sun.com/__bugdatabase/view_bug.do?bug___id=7201156
<http://bugs.sun.com/bugdatabase/view_bug.do?bug_id=7201156>
webrev: http://cr.openjdk.java.net/~__zhouyx/7201156/webrev.00/
<http://cr.openjdk.java.net/~zhouyx/7201156/webrev.00/>
--
Best Regards,
Sean Chou