diff options
author | Tor Lillqvist <tml@novell.com> | 2005-03-20 01:51:38 +0000 |
---|---|---|
committer | Tor Lillqvist <tml@src.gnome.org> | 2005-03-20 01:51:38 +0000 |
commit | 336a76e8ea141f4f9361c708a4778e5c377cd407 (patch) | |
tree | a0c2ad9480b2d338709b1701f6d9a6cf3bb1f434 /pango | |
parent | c6389d30c6dd3208217d0148c6f240b2dcbba945 (diff) | |
download | pango-336a76e8ea141f4f9361c708a4778e5c377cd407.tar.gz |
Force fRTL for each item returned from ScriptItemize() to the bidi
2005-03-20 Tor Lillqvist <tml@novell.com>
* modules/basic/basic-win32.c (itemize_shape_and_place): Force
fRTL for each item returned from ScriptItemize() to the bidi
embedding parity for the whole run. Pango passes us a run that it
correctly has analyzed should be of a certain direction, but
ScriptItemize() still might split it into items, some of which
have neutral direction. (#170931)
(uniscribe_shape, text_is_simple): Use g_utf8_to_utf16().
Merge from HEAD:
* pango/pangowin32-private.h
* pango/pangowin32.c (pango_win32_font_get_metrics): Cache metrics
by language, like the other backends do. Don't try to get the
width of the 0..9 digit string if the font doesn't cover digits.
* modules/basic/Makefile.am (INCLUDES): Remove extraneous slash
that confuses gcc on Win32.
Diffstat (limited to 'pango')
-rw-r--r-- | pango/pangowin32-private.h | 2 | ||||
-rw-r--r-- | pango/pangowin32.c | 137 |
2 files changed, 93 insertions, 46 deletions
diff --git a/pango/pangowin32-private.h b/pango/pangowin32-private.h index b1014214..e51f2d55 100644 --- a/pango/pangowin32-private.h +++ b/pango/pangowin32-private.h @@ -75,6 +75,8 @@ struct _PangoWin32Font LOGFONT logfont; int size; + GSList *metrics_by_lang; + PangoFontMap *fontmap; /* Written by pango_win32_get_hfont: */ diff --git a/pango/pangowin32.c b/pango/pangowin32.c index 0a02dbc2..62af6bdd 100644 --- a/pango/pangowin32.c +++ b/pango/pangowin32.c @@ -45,6 +45,14 @@ HDC pango_win32_hdc; OSVERSIONINFO pango_win32_os_version_info; gboolean pango_win32_debug = FALSE; +typedef struct _PangoWin32MetricsInfo PangoWin32MetricsInfo; + +struct _PangoWin32MetricsInfo +{ + const char *sample_str; + PangoFontMetrics *metrics; +}; + typedef struct _PangoWin32FontClass PangoWin32FontClass; struct _PangoWin32FontClass @@ -145,6 +153,8 @@ pango_win32_font_init (PangoWin32Font *win32font) { win32font->size = -1; + win32font->metrics_by_lang = NULL; + win32font->glyph_info = g_hash_table_new_full (NULL, NULL, NULL, g_free); } @@ -535,59 +545,84 @@ static PangoFontMetrics * pango_win32_font_get_metrics (PangoFont *font, PangoLanguage *language) { - HFONT hfont; - TEXTMETRIC tm; - PangoLayout *layout; - PangoRectangle extents; - PangoContext *context; - PangoFontMetrics *metrics; - - metrics = pango_font_metrics_new (); - - metrics->ascent = 0; - metrics->descent = 0; - metrics->approximate_digit_width = 0; - metrics->approximate_char_width = 0; + PangoWin32MetricsInfo *info = NULL; /* Quiet gcc */ + PangoWin32Font *win32font = (PangoWin32Font *)font; + GSList *tmp_list; + + const char *sample_str = pango_language_get_sample_string (language); - hfont = pango_win32_get_hfont (font); + tmp_list = win32font->metrics_by_lang; + while (tmp_list) + { + info = tmp_list->data; + + if (info->sample_str == sample_str) /* We _don't_ need strcmp */ + break; - if (hfont != NULL) + tmp_list = tmp_list->next; + } + + if (!tmp_list) { - PangoFontDescription *font_desc; + HFONT hfont; + PangoFontMetrics *metrics; + + info = g_new (PangoWin32MetricsInfo, 1); + win32font->metrics_by_lang = g_slist_prepend (win32font->metrics_by_lang, info); + + info->sample_str = sample_str; + info->metrics = metrics = pango_font_metrics_new (); + + hfont = pango_win32_get_hfont (font); + if (hfont != NULL) + { + PangoCoverage *coverage; + TEXTMETRIC tm; - SelectObject (pango_win32_hdc, hfont); - GetTextMetrics (pango_win32_hdc, &tm); + SelectObject (pango_win32_hdc, hfont); + GetTextMetrics (pango_win32_hdc, &tm); - metrics->ascent = tm.tmAscent * PANGO_SCALE; - metrics->descent = tm.tmDescent * PANGO_SCALE; - metrics->approximate_char_width = tm.tmAveCharWidth * PANGO_SCALE; + metrics->ascent = tm.tmAscent * PANGO_SCALE; + metrics->descent = tm.tmDescent * PANGO_SCALE; + metrics->approximate_char_width = tm.tmAveCharWidth * PANGO_SCALE; - /* lovely copy&paste programming (from pangox.c) FIXME: This is - * sort of a sledgehammer solution, if we cached the results, - * like the other backends did, it wouldn't big a huge performance - * problem. Get the avg. width of the chars in "0123456789" - */ - context = pango_win32_get_context (); - pango_context_set_language (context, language); - font_desc = pango_font_describe (font); - pango_context_set_font_description (context, font_desc); - layout = pango_layout_new (context); - pango_layout_set_text (layout, "0123456789", -1); - metrics->approximate_digit_width = max_glyph_width (layout); - - /* FIXME: Should get the real values from the TrueType font file */ - metrics->underline_position = -2 * PANGO_SCALE; - metrics->underline_thickness = 1 * PANGO_SCALE; - metrics->strikethrough_thickness = metrics->underline_thickness; - /* Really really wild guess */ - metrics->strikethrough_position = metrics->ascent / 3; - - pango_font_description_free (font_desc); - g_object_unref (layout); - g_object_unref (context); - } + coverage = pango_win32_font_get_coverage (font, language); + if (pango_coverage_get (coverage, '0') != PANGO_COVERAGE_NONE && + pango_coverage_get (coverage, '9') != PANGO_COVERAGE_NONE) + { + PangoContext *context; + PangoFontDescription *font_desc; + PangoLayout *layout; + + /* Get the average width of the chars in "0123456789" */ + context = pango_win32_get_context (); + pango_context_set_language (context, language); + font_desc = pango_font_describe (font); + pango_context_set_font_description (context, font_desc); + layout = pango_layout_new (context); + pango_layout_set_text (layout, "0123456789", -1); + + metrics->approximate_digit_width = max_glyph_width (layout); + + pango_font_description_free (font_desc); + g_object_unref (layout); + g_object_unref (context); + } + else + metrics->approximate_digit_width = metrics->approximate_char_width; + + pango_coverage_unref (coverage); - return metrics; + /* FIXME: Should get the real values from the TrueType font file */ + metrics->underline_position = -2 * PANGO_SCALE; + metrics->underline_thickness = 1 * PANGO_SCALE; + metrics->strikethrough_thickness = metrics->underline_thickness; + /* Really really wild guess */ + metrics->strikethrough_position = metrics->ascent / 3; + } + } + + return pango_font_metrics_ref (info->metrics); } @@ -631,6 +666,13 @@ pango_win32_font_dispose (GObject *object) } static void +free_metrics_info (PangoWin32MetricsInfo *info) +{ + pango_font_metrics_unref (info->metrics); + g_free (info); +} + +static void pango_win32_font_finalize (GObject *object) { PangoWin32Font *win32font = (PangoWin32Font *)object; @@ -639,6 +681,9 @@ pango_win32_font_finalize (GObject *object) if (win32font->hfont != NULL) pango_win32_font_cache_unload (cache, win32font->hfont); + g_slist_foreach (win32font->metrics_by_lang, (GFunc)free_metrics_info, NULL); + g_slist_free (win32font->metrics_by_lang); + if (win32font->win32face) pango_win32_font_entry_remove (win32font->win32face, PANGO_FONT (win32font)); |