Looks fine.

-Joe

On 8/8/2011 4:06 PM, Mandy Chung wrote:
Sherman,

Thanks.  Can you review one more fix:

7012365: TEST_BUG: test/java/nio/charset/spi/basic.sh can be run with Cygwin
Webrev at:
  http://cr.openjdk.java.net/~mchung/7012365/webrev.00/

Mandy

On 8/8/11 3:16 PM, Xueming Shen wrote:
Looks good. Thanks for fixing them.

On 08/08/2011 02:37 PM, Mandy Chung wrote:
Simple fix on these tests to run on cygwin.

7036518: TEST_BUG: add cygwin support to
test/java/nio/charset/coders/CheckSJISMappingProp.sh
7036519: TEST_BUG: add cygwin support to test/demo/zipfs/basic.sh

Webrev at:
      http://cr.openjdk.java.net/~mchung/7036518/webrev.00/

Thanks
Mandy

Reply via email to