On 2/19/2015 8:57 PM, Phil Race wrote:
On 2/19/15 8:50 PM, Mandy Chung wrote:
On 2/19/2015 4:23 PM, Phil Race wrote:
http://cr.openjdk.java.net/~prr/8035302/
I'll let Sherman and others to do the detailed review here. Minor
comment:
FontDescriptor.java
line 56-58: can use try-with-resource.
Its a ByteArrayInputStream there is no leak potential
line 60 catches my attention even no leak potential.
line 62: should it throw UncheckedIOException?
Completely theoretical as far as I can tell.
That's right.
With this change, for the benefit of jigsaw, the Charset support
needed by the font code
now uses just the public APIs.
java.desktop to the internal of charsets is an undesirable edge in
the module graph [1]. Thank you for eliminating it. It's not ideal
to copy sun.nio code while some day these sun.awt.motif.** will no
longer be needed and can be removed.
With this dependency removed, <top-repo>/modules.xml needs to be
updated. Here is the patch:
I wondered who would do this but if you want it part of this patch, no
problem I can include it.
Yes please and make verify-modules to verify the dependencies.
Mandy
