RFR: 8349008: Remove temporary font file tracking code [v2]

Lukasz Kostyra lkostyra at openjdk.org
Mon Feb 3 10:18:30 UTC 2025


> This is a cleanup follow-up, removing `FontFileWriter.FontTracker` and all related uses. `FontTracker` was tracking font size use when SecurityManager was present, however since we removed SM, `FontTracker` was no longer activated and as such was dead code.
> 
> `FontFileWriter.FontTracker` and its use in `FontFileWriter` + related methods were removed. This in turn cleaned up `PrismFontFile` and made a couple of variables not longer used, including a `bool tracking` argument in constructor. These cleanups propagated to `PrismFontFactory`, `{CT,DW,FT}Factory` and `{CT,DW,FT}FontFile` classes.
> 
> Tests worked the same after this change.

Lukasz Kostyra has updated the pull request incrementally with one additional commit since the last revision:

  PrismFontFactory review fixes
  
  - Updated copyright header year
  - Fixed comment mentioning tracker

-------------

Changes:
  - all: https://git.openjdk.org/jfx/pull/1693/files
  - new: https://git.openjdk.org/jfx/pull/1693/files/27d8f8f9..61c51bb8

Webrevs:
 - full: https://webrevs.openjdk.org/?repo=jfx&pr=1693&range=01
 - incr: https://webrevs.openjdk.org/?repo=jfx&pr=1693&range=00-01

  Stats: 3 lines in 1 file changed: 0 ins; 1 del; 2 mod
  Patch: https://git.openjdk.org/jfx/pull/1693.diff
  Fetch: git fetch https://git.openjdk.org/jfx.git pull/1693/head:pull/1693

PR: https://git.openjdk.org/jfx/pull/1693


More information about the openjfx-dev mailing list