[gtk/fontchooser-fix-3] fontchooser: Fix axis name handling
- From: Matthias Clasen <matthiasc src gnome org>
- To: commits-list gnome org
- Cc:
- Subject: [gtk/fontchooser-fix-3] fontchooser: Fix axis name handling
- Date: Fri, 7 Jan 2022 19:28:25 +0000 (UTC)
commit b4eeada025140afa2c5b4a59cdfe2abce132d7e8
Author: Matthias Clasen <mclasen redhat com>
Date: Fri Jan 7 14:27:17 2022 -0500
fontchooser: Fix axis name handling
This code was just broken.
Fixes: #4603
gtk/gtkfontchooserwidget.c | 20 ++++++++++----------
1 file changed, 10 insertions(+), 10 deletions(-)
---
diff --git a/gtk/gtkfontchooserwidget.c b/gtk/gtkfontchooserwidget.c
index b4b8952cb1..43ba0e9d27 100644
--- a/gtk/gtkfontchooserwidget.c
+++ b/gtk/gtkfontchooserwidget.c
@@ -1621,12 +1621,13 @@ static struct {
* We actually don't bother about the FT_Face here, but we use this so that we can have a single
* version of add_axis() taylored to PangoFT2 or Pango with HarfBuzz integrated
*/
-static void *
+static void
get_font_name (FT_Face face,
FT_Var_Axis *ax,
- const char *result)
+ char *buffer,
+ guint buffer_len)
{
- result = ax->name;
+ g_strlcpy (buffer, ax->name, buffer_len);
}
static const float
@@ -1661,13 +1662,10 @@ get_axis_float_default (FT_Var_Axis *ax)
static void
get_font_name (hb_face_t *face,
hb_ot_var_axis_info_t *ax,
- const char *name)
+ char *buffer,
+ unsigned int buffer_len)
{
- char buffer[20];
- unsigned int buffer_len = 20;
-
hb_ot_name_get_utf8 (face, ax->name_id, HB_LANGUAGE_INVALID, &buffer_len, buffer);
- name = buffer;
}
#define get_float_value(x) x
@@ -1712,7 +1710,8 @@ add_axis (GtkFontChooserWidget *fontchooser,
{
GtkFontChooserWidgetPrivate *priv = fontchooser->priv;
Axis *axis;
- const char *name;
+ char buffer[20];
+ char *name;
int i;
axis = g_new (Axis, 1);
@@ -1720,7 +1719,8 @@ add_axis (GtkFontChooserWidget *fontchooser,
axis->fontchooser = GTK_WIDGET (fontchooser);
axis->default_value = get_axis_float_default (ax);
- get_font_name (face, ax, name);
+ get_font_name (face, ax, buffer, 20);
+ name = buffer;
for (i = 0; i < G_N_ELEMENTS (axis_names); i++)
{
[
Date Prev][
Date Next] [
Thread Prev][
Thread Next]
[
Thread Index]
[
Date Index]
[
Author Index]