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

Andy Goryachev angorya at openjdk.org
Mon Feb 3 15:11:54 UTC 2025


On Mon, 3 Feb 2025 10:18:30 GMT, Lukasz Kostyra <lkostyra at openjdk.org> wrote:

>> 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

Marked as reviewed by angorya (Reviewer).

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

PR Review: https://git.openjdk.org/jfx/pull/1693#pullrequestreview-2590212996


More information about the openjfx-dev mailing list