[9] RFR (XXS): 8035186: j2se_jdk/jdk/test/java/lang/invoke/lambda/LogGeneratedClassesTest.java - assertion error
Paul Sandoz
paul.sandoz at oracle.com
Wed May 28 10:04:38 UTC 2014
On May 27, 2014, at 6:16 PM, Vladimir Ivanov <vladimir.x.ivanov at oracle.com> wrote:
> Good catch!
> One more iteration:
> http://cr.openjdk.java.net/~vlivanov/8035186/webrev.02/
>
Looks good.
Some minor points, no need for another review round either way:
- don't need the nested try blocks in isWriteableDirectory, just tack the finally block on the end of the last catch block.
- could add a println when there is no POSIX environment to indicate the test is not run and remove it when/if jtreg supports skipping.
Paul.
More information about the core-libs-dev
mailing list