mirrors/emacs
1
Fork 0
mirror of git://git.sv.gnu.org/emacs.git synced 2025-12-25 23:10:47 -08:00
Code Issues Projects Releases Wiki Activity

Avert a crash in the Android last resort font driver

Browse source 
* java/org/gnu/emacs/EmacsFontDriver.java (FONT_INVALID_CODE):
New constant.
(hasChar, encodeChar): Accept a jint character code rather than
a jchar.

* java/org/gnu/emacs/EmacsSdk23FontDriver.java (hasChar)
(encodeChar): Return 0 or FONT_INVALID_CODE if the character is
not representable in a Java char.

* java/org/gnu/emacs/EmacsSdk7FontDriver.java (hasChar): Ditto.

* src/androidfont.c (android_init_font_driver): Adjust signature
to match.
This commit is contained in:
Po Lu 2023-09-14 16:53:04 +08:00
parent f13b08be11
commit 6ffb0a1719
4 changed files with 31 additions and 10 deletions
Download patch file Download diff file Expand all files Collapse all files

11
java/org/gnu/emacs/EmacsFontDriver.java
View file

@ -63,6 +63,11 @@ public abstract class EmacsFontDriver
public static final int MONO = 100;
public static final int CHARCELL = 110;
/* Special glyph codes. */
public static final int FONT_INVALID_CODE = 0xFFFFFFFF;
public static class FontSpec
{
/* The fields below mean the same as they do in enum
@ -148,15 +153,17 @@ public abstract class EmacsFontDriver
}
};
/* These mean the same as they do in struct font_driver. */
public abstract FontEntity[] list (FontSpec fontSpec);
public abstract FontEntity match (FontSpec fontSpec);
public abstract String[] listFamilies ();
public abstract FontObject openFont (FontEntity fontEntity, int pixelSize);
public abstract int hasChar (FontSpec font, char charCode);
public abstract int hasChar (FontSpec font, int charCode);
public abstract void textExtents (FontObject font, int code[],
FontMetrics fontMetrics);
public abstract int encodeChar (FontObject fontObject, char charCode);
public abstract int encodeChar (FontObject fontObject, int charCode);
public abstract int draw (FontObject fontObject, EmacsGC gc,
EmacsDrawable drawable, int[] chars,
int x, int y, int backgroundWidth,

10
java/org/gnu/emacs/EmacsSdk23FontDriver.java
View file

@ -96,7 +96,7 @@ public final class EmacsSdk23FontDriver extends EmacsSdk7FontDriver
@Override
public int
hasChar (FontSpec font, char charCode)
hasChar (FontSpec font, int charCode)
{
Sdk7FontObject fontObject;
Paint paint;
@ -109,6 +109,12 @@ public final class EmacsSdk23FontDriver extends EmacsSdk7FontDriver
else
paint = ((Sdk7FontEntity) font).typeface.typefacePaint;
return paint.hasGlyph (String.valueOf (charCode)) ? 1 : 0;
/* If the character falls within the confines of the BMP, return
1. */
if (charCode < 65536)
return paint.hasGlyph (String.valueOf ((char) charCode)) ? 1 : 0;
/* Otherwise return 0. */
return 0;
}
};

16
java/org/gnu/emacs/EmacsSdk7FontDriver.java
View file

@ -291,13 +291,18 @@ public class EmacsSdk7FontDriver extends EmacsFontDriver
@Override
public int
hasChar (FontSpec font, char charCode)
hasChar (FontSpec font, int charCode)
{
float missingGlyphWidth, width;
Rect rect1, rect2;
Paint paint;
Sdk7FontObject fontObject;
/* Ignore characters outside the BMP. */
if (charCode > 65535)
return 0;
if (font instanceof Sdk7FontObject)
{
fontObject = (Sdk7FontObject) font;
@ -308,7 +313,7 @@ public class EmacsSdk7FontDriver extends EmacsFontDriver
paint.setTextSize (10);
if (Character.isWhitespace (charCode))
if (Character.isWhitespace ((char) charCode))
return 1;
missingGlyphWidth = paint.measureText (TOFU_STRING);
@ -325,7 +330,7 @@ public class EmacsSdk7FontDriver extends EmacsFontDriver
paint.getTextBounds (TOFU_STRING, 0, TOFU_STRING.length (),
rect1);
paint.getTextBounds ("" + charCode, 0, 1, rect2);
paint.getTextBounds ("" + (char) charCode, 0, 1, rect2);
return rect1.equals (rect2) ? 0 : 1;
}
@ -410,8 +415,11 @@ public class EmacsSdk7FontDriver extends EmacsFontDriver
@Override
public int
encodeChar (FontObject fontObject, char charCode)
encodeChar (FontObject fontObject, int charCode)
{
if (charCode > 65535)
return FONT_INVALID_CODE;
return charCode;
}