<AWT Dev> RfR JDK-8055160

[Pete Brunet]

Thanks Mandy, The new patch is at
http://cr.openjdk.java.net/~ptbrunet/JDK-8055160/webrev.04

On 6/10/15 6:29 PM, Mandy Chung wrote:
>> 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.
Done, using UncheckedIOException.
>
> 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.
Done.  I was removing them in the shell script but I've moved that now
to the Java code.  I had to add delete to the permissions in
accessibilityProvider.sp.
>
> Otherwise looks good to me.
>
> make/Images.gmk needs fixing as I mentioned earlier.
Done in JDK-8078335.
>
> Mandy
>
>