> On Jun 10, 2015, at 3:33 PM, Pete Brunet <[email protected]> wrote:
>
> Due to some other priorities it's been over 2 months since the last webrev.
> An update is here:
> http://cr.openjdk.java.net/~ptbrunet/JDK-8055160/webrev.03
BarProvider.java
You can use try-with-resource
try (PrintWriter writer = new PrintWriter("BarProvider.txt”)) {
writer.println("BarProvider-activated”);
} catch (IOException e) {
throw new UncheckedException(e);
}
That will close the writer. I think you should rethow IOException instead of
swallowing it.
Same comment to other provider implementation.
Load.java
The test should clean up all files before starting loading the providers
since someone may run the test standalone multiple times on the same directory.
Otherwise looks good to me.
make/Images.gmk needs fixing as I mentioned earlier.
Mandy
