]> code.delx.au - gnu-emacs/commitdiff
Remove unneeded workaround in xftfont.c
authorEli Zaretskii <eliz@gnu.org>
Thu, 25 Feb 2016 19:59:57 +0000 (21:59 +0200)
committerEli Zaretskii <eliz@gnu.org>
Thu, 25 Feb 2016 19:59:57 +0000 (21:59 +0200)
* src/xftfont.c (xftfont_open): Remove "dirty workaround" for
XftTextExtents8 behavior, as it is no longer needed.  Suggested by
Fangwen Yu <yynyygy@gmail.com>.  (Bug#22383)

src/xftfont.c

index 110f99a9be94d29169517ec83915f5ac6a2403f9..d94955f296a05da1b4f9d22e683cfb84698fd61b 100644 (file)
@@ -395,16 +395,6 @@ xftfont_open (struct frame *f, Lisp_Object entity, int pixel_size)
 
   font->ascent = xftfont->ascent;
   font->descent = xftfont->descent;
-  if (pixel_size >= 5)
-    {
-      /* The above condition is a dirty workaround because
-        XftTextExtents8 behaves strangely for some fonts
-        (e.g. "Dejavu Sans Mono") when pixel_size is less than 5. */
-      if (font->ascent < extents.y)
-       font->ascent = extents.y;
-      if (font->descent < extents.height - extents.y)
-       font->descent = extents.height - extents.y;
-    }
   font->height = font->ascent + font->descent;
 
   if (XINT (AREF (entity, FONT_SIZE_INDEX)) == 0)