On Mon, 19 Jul 2021 09:38:27 GMT, Maxim Kartashev
<[email protected]> wrote:
>> Added an `ExceptionCheck()` followed by `ExceptionDescribe()` and
>> `ExceptionClear()` immediately after the Java calls made from the callback
>> function `ReadTTFontFileFunc()` in `freetypeScaler.c`.
>>
>> The exception(s) need to be cleared because we're not returning immediately
>> to Java that would've been able to handle them gracefully. And in order not
>> to loose the exception entirely (even though the return value would also
>> indicate an error condition), print out the exception with
>> `ExceptionDescribe()` to aid in debugging.
>
> Maxim Kartashev has updated the pull request incrementally with one
> additional commit since the last revision:
>
> Addressed PR comments
>
> 1. Added CHECK_NULL() to awt_Component.cpp
test/jdk/java/awt/font/JNICheck/FreeTypeScalerJNICheck.java line 29:
> 27: * @summary Verifies that -Xcheck:jni issues no warnings from
> freetypeScaler.c
> 28: * @library /test/lib
> 29: * @key headful
What about this test is headful ?
test/jdk/java/awt/font/JNICheck/FreeTypeScalerJNICheck.java line 36:
> 34: import java.awt.geom.Rectangle2D;
> 35: import java.awt.image.*;
> 36: import java.io.*;
Can we get rid of all these wild card imports ?
test/jdk/java/awt/font/JNICheck/FreeTypeScalerJNICheck.java line 59:
> 57: for (String ff : families)
> 58: {
> 59: Font font = Font.decode(ff);
Gosh, does anyone still use decode() ? I keep forgetting it exists.
You have all the family names, why not just new Font(ff, Font.PLAIN, 12) ?
-------------
PR: https://git.openjdk.java.net/jdk/pull/4572
