Hi, Phil. > The fix has been updated. > http://cr.openjdk.java.net/~prr/8039273.1/ > 2nd reviewer anyone ?
Looks fine. I assume that SecurityException from File.exists() is expected, or the code is executed under doPriviliged() block. > > -phil. > > > On 12/09/2016 09:44 AM, Naoto Sato wrote: >> Hi Phil, >> >> In FontConfiguration.java, line 193-195 can be inserted just after line 191, >> as fontConfigFile is guaranteed to be non-null if userConfigFile is non >> null. Otherwise it looks good to me. >> >> Naoto >> >> On 12/8/16 12:01 PM, Phil Race wrote: >>> Bug: https://bugs.openjdk.java.net/browse/JDK-8039273 >>> Webrev : http://cr.openjdk.java.net/~prr/8039273/ >>> >>> A CCC request and documentation are being prepared. >>> >>> -phil >
