Drop no longer needed glyph_widths cache

git-svn-id: svn://tug.org/texlive/trunk@29388 c570f23f-e606-0410-a88d-b1316a301751

4 files changed, 13 insertions, 32 deletions

diff --git a/Build/source/texk/xdvipdfmx/ChangeLog b/Build/source/texk/xdvipdfmx/ChangeLog
index 73e749b54bc..99d540390c7 100644
--- a/Build/source/texk/xdvipdfmx/ChangeLog
+++ b/Build/source/texk/xdvipdfmx/ChangeLog
@@ -11,6 +11,8 @@
 	 for XDV_FLAG_FONTTYPE_* flags as they do nothing.
 	 * src/dvi.c (do_glyph_array): Use FT_Get_Advance() instead of the slower
 	 FT_Load_Glyph().
+	 * src/dvi.c, src/fontmap.c, src/fontmap.h: Drop no longer needed
+	 glyph_widths cache.
 
 2013-02-25  Peter Breitenlohner  <peb@mppmu.mpg.de>
 
diff --git a/Build/source/texk/xdvipdfmx/src/dvi.c b/Build/source/texk/xdvipdfmx/src/dvi.c
index c713732cce3..e4f56d73969 100644
--- a/Build/source/texk/xdvipdfmx/src/dvi.c
+++ b/Build/source/texk/xdvipdfmx/src/dvi.c
@@ -128,7 +128,6 @@ static struct loaded_font
 #ifdef XETEX
   unsigned long rgba_color;
   FT_Face ft_face;
-  unsigned short *glyph_widths;
   int   layout_dir;
   float extend;
   float slant;
@@ -1063,14 +1062,6 @@ dvi_locate_native_font (const char *ps_name,
   free(fontmap_key);
 
   loaded_fonts[cur_id].ft_face = mrec->opt.ft_face;
-  if (mrec->opt.glyph_widths == NULL) {
-    /* this is the first loaded_font that refers to this fontmap record,
-       so allocate the glyph_width cache and save it in the fontmap */
-    mrec->opt.glyph_widths = NEW(mrec->opt.ft_face->num_glyphs, unsigned short);
-    for (i = 0; i < mrec->opt.ft_face->num_glyphs; ++i)
-      mrec->opt.glyph_widths[i] = 0xffff;
-  }
-  loaded_fonts[cur_id].glyph_widths = mrec->opt.glyph_widths;
   loaded_fonts[cur_id].layout_dir = layout_dir;
   loaded_fonts[cur_id].extend = mrec->opt.extend;
   loaded_fonts[cur_id].slant = mrec->opt.slant;
@@ -1935,22 +1926,18 @@ do_glyph_array (int yLocsPresent)
     glyph_id = get_buffered_unsigned_pair(); /* freetype glyph index */
 
     if (glyph_id < font->ft_face->num_glyphs) {
-      if (font->glyph_widths[glyph_id] == 0xffff) {
-        FT_Error error;
-        FT_Fixed advance;
-        int flags = FT_LOAD_NO_SCALE;
-
-        if (font->layout_dir == 1)
-          flags |= FT_LOAD_VERTICAL_LAYOUT;
-
-        error = FT_Get_Advance(font->ft_face, glyph_id, flags, &advance);
-        if (error)
-          font->glyph_widths[glyph_id] = 0;
-        else
-          font->glyph_widths[glyph_id] = advance;
-      }
+      FT_Error error;
+      FT_Fixed advance;
+      int flags = FT_LOAD_NO_SCALE;
+
+      if (font->layout_dir == 1)
+        flags |= FT_LOAD_VERTICAL_LAYOUT;
+
+      error = FT_Get_Advance(font->ft_face, glyph_id, flags, &advance);
+      if (error)
+        advance = 0;
 
-      glyph_width = (double)font->size * (double)font->glyph_widths[glyph_id] / (double)font->ft_face->units_per_EM;
+      glyph_width = (double)font->size * (double)advance / (double)font->ft_face->units_per_EM;
       glyph_width = glyph_width * font->extend;
       if (compute_boxes && link_annot && marked_depth >= tagged_depth) {
         pdf_rect rect;
diff --git a/Build/source/texk/xdvipdfmx/src/fontmap.c b/Build/source/texk/xdvipdfmx/src/fontmap.c
index 12957ddda30..044dea7d803 100644
--- a/Build/source/texk/xdvipdfmx/src/fontmap.c
+++ b/Build/source/texk/xdvipdfmx/src/fontmap.c
@@ -86,7 +86,6 @@ pdf_init_fontmap_record (fontmap_rec *mrec)
 
 #ifdef XETEX
   mrec->opt.ft_face   = NULL;
-  mrec->opt.glyph_widths = NULL;
 #endif
 }
 
@@ -112,10 +111,6 @@ pdf_clear_fontmap_record (fontmap_rec *mrec)
     RELEASE(mrec->opt.otl_tags);
   if (mrec->opt.charcoll)
     RELEASE(mrec->opt.charcoll);
-#ifdef XETEX
-  if (mrec->opt.glyph_widths)
-    RELEASE(mrec->opt.glyph_widths);
-#endif
   pdf_init_fontmap_record(mrec);
 }
 
@@ -160,7 +155,6 @@ pdf_copy_fontmap_record (fontmap_rec *dst, const fontmap_rec *src)
 
 #ifdef XETEX
   dst->opt.ft_face   = src->opt.ft_face;
-  dst->opt.glyph_widths = src->opt.glyph_widths;
 #endif
 }
 
@@ -1077,7 +1071,6 @@ pdf_insert_native_fontmap_record (const char *name, const char *path, int index,
   mrec->font_name = (path != NULL) ? mstrdup(path) : NULL;
   mrec->opt.index = index;
   mrec->opt.ft_face = face;
-  mrec->opt.glyph_widths = NULL;
   if (layout_dir != 0)
     mrec->opt.flags |= FONTMAP_OPT_VERT;
 
diff --git a/Build/source/texk/xdvipdfmx/src/fontmap.h b/Build/source/texk/xdvipdfmx/src/fontmap.h
index c257e692c78..fee6f238fa5 100644
--- a/Build/source/texk/xdvipdfmx/src/fontmap.h
+++ b/Build/source/texk/xdvipdfmx/src/fontmap.h
@@ -60,7 +60,6 @@ typedef struct fontmap_opt {
   int    stemv;       /* StemV value especially for CJK fonts */
 #ifdef XETEX
   FT_Face ft_face;
-  unsigned short *glyph_widths;
 #endif
 } fontmap_opt;