On Fri, 18 Sep 2020 20:23:25 GMT, Phil Race <p...@openjdk.org> wrote:
> I changed it to read the file into a ByteBuffer to avoid the problem that we > can't control when NIO free's the mmaped > buffer. This is a problem for leaking tmp Type1 font files when using > createFont and I thought maybe we could just > change that case, but Type 1 fonts are getting very rare. Neither Mac nor > Windows ship them and Mac at least does not > support them at all, and current Linuxes seem to be almost all TTF and OTF > fonts. Also T1 fonts are small, so any > minimal peformance benefit is not worth it. There isn't a specific regression > test. I verified this on Ubuntu 20.04 > which has at least a (very) few Type 1 fonts. This pull request has now been integrated. Changeset: cc7521c4 Author: Phil Race <p...@openjdk.org> URL: https://git.openjdk.java.net/jdk/commit/cc7521c4 Stats: 13 lines in 1 file changed: 1 ins; 1 del; 11 mod 8252199: Reimplement support of Type 1 fonts without MappedByteBuffer Reviewed-by: serb, jdv ------------- PR: https://git.openjdk.java.net/jdk/pull/258
