RFR: 8344114: Remove obsolete permission check methods from Font classes
Lukasz Kostyra
lkostyra at openjdk.org
Mon Dec 9 12:48:47 UTC 2024
On Fri, 6 Dec 2024 16:51:48 GMT, Kevin Rushforth <kcr at openjdk.org> wrote:
>> This PR removes obsolete permission check methods and fields from Font-related classes.
>>
>> Verified the changes on Windows, did not see any regressions.
>
> modules/javafx.graphics/src/main/java/com/sun/javafx/font/PrismFontFactory.java line 1375:
>
>> 1373: if (FontFileWriter.hasTempPermission()) {
>> 1374: return loadEmbeddedFont0(name, fontStream, size, register, loadAll);
>> 1375: }
>
> This test always returns true today, so it looks like you removed the code path that was in use. Unless I'm missing something, I think that you should restore the "return loadEmbeddedFont0" call and remove everything else below this point from this method.
Yes, this should always `return loadEmbeddedFont0`. Again, some Friday-related mishap.
-------------
PR Review Comment: https://git.openjdk.org/jfx/pull/1659#discussion_r1875926343
More information about the openjfx-dev
mailing list