<AWT Dev> RfR JDK-8055160
Mandy Chung
mandy.chung at oracle.com
Wed Jun 10 23:29:33 UTC 2015
> On Jun 10, 2015, at 3:33 PM, Pete Brunet <peter.brunet at oracle.com> 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
More information about the build-dev
mailing list