On Mon, 3 Feb 2025 10:18:30 GMT, Lukasz Kostyra <[email protected]> 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 kcr (Lead).
-------------
PR Review: https://git.openjdk.org/jfx/pull/1693#pullrequestreview-2589926704
