Code review: 6339649

2010-08-17 Thread Michael McMahon
Chris, Could you review the following small change? http://cr.openjdk.java.net/~michaelm/6339649/webrev.1/ I think it's a useful change, since many applications won't bother to even catch the runtime exception except at a fairly generic high level and it is useful (in this case) to have the mes

Re: Code review: 6339649

2010-08-17 Thread Chris Hegarty
Michael, This change looks fine. -Chris Michael McMahon wrote: Chris, Could you review the following small change? http://cr.openjdk.java.net/~michaelm/6339649/webrev.1/ I think it's a useful change, since many applications won't bother to even catch the runtime exception except at a fairly

hg: jdk7/tl/jdk: 6339649: URI.create should include a detail message when throwing IllegalArgumentException

2010-08-17 Thread michael . x . mcmahon
Changeset: 42eaa082a4e6 Author:michaelm Date: 2010-08-17 14:49 +0100 URL: http://hg.openjdk.java.net/jdk7/tl/jdk/rev/42eaa082a4e6 6339649: URI.create should include a detail message when throwing IllegalArgumentException Summary: create enclosing exception with message of enclosed

hg: jdk7/tl/jdk: 6969651: TEST_BUG: tools/jar/JarEntryTime.java failed on JDK7 when run on NFS

2010-08-17 Thread xueming . shen
Changeset: bfc3855a9341 Author:sherman Date: 2010-08-17 16:01 -0700 URL: http://hg.openjdk.java.net/jdk7/tl/jdk/rev/bfc3855a9341 6969651: TEST_BUG: tools/jar/JarEntryTime.java failed on JDK7 when run on NFS Summary: changed to use more appropriate nfs file time Reviewed-by: martin