Request for review: 7201156 : jar tool fails to convert file separation characters for list and extract

Chris Hegarty chris.hegarty at oracle.com
Fri Nov 9 09:57:03 UTC 2012


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.hegarty at oracle.com
> <mailto:chris.hegarty at 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
>



More information about the core-libs-dev mailing list