[prev in list] [next in list] [prev in thread] [next in thread] 

List:       openjdk-2d-dev
Subject:    [OpenJDK 2D-Dev] Integrated: 8269223: -Xcheck:jni WARNINGs working with fonts on Linux
From:       Maxim Kartashev <github.com+28651297+mkartashev () openjdk ! java ! net>
Date:       2021-08-23 20:31:41
Message-ID: OIdA8uFDFOexNDpjQ9XiXksDu0n8gZmmHgiC78Rf-fs=.a5812d47-f55f-4b22-970e-c4d6cd2b1011 () github ! com
[Download RAW message or body]

On Wed, 23 Jun 2021 09:19:16 GMT, Maxim Kartashev \
<github.com+28651297+mkartashev@openjdk.org> 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.

This pull request has now been integrated.

Changeset: 9bc02322
Author:    Maxim Kartashev <maxim.kartashev@jetbrains.com>
Committer: Sergey Bylokhov <serb@openjdk.org>
URL:       https://git.openjdk.java.net/jdk/commit/9bc023220fbbb0b6ea1ed1a0ca2aa3848764e8cd
                
Stats:     105 lines in 5 files changed: 98 ins; 0 del; 7 mod

8269223: -Xcheck:jni WARNINGs working with fonts on Linux

Reviewed-by: prr, serb

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

PR: https://git.openjdk.java.net/jdk/pull/4572


[prev in list] [next in list] [prev in thread] [next in thread]