[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

bug#2994: Recent change to font.c:font_load_for_lface affects variable-p

From: Juanma Barranquero
Subject: bug#2994: Recent change to font.c:font_load_for_lface affects variable-pitch face in the modeline
Date: Tue, 14 Apr 2009 13:57:12 +0200

Package: emacs
X-Debbugs-CC: <address@hidden>

This change affects using variable-pitch on the modeline:

2009-04-13  Kenichi Handa  <address@hidden>

        * font.c [...]
        (font_load_for_lface): Get a font that supports at least ASCII

@@ -3455,7 +3461,8 @@ font_load_for_lface (f, attrs, spec)
   Lisp_Object entity;

-  entity = font_find_for_lface (f, attrs, spec, -1);
+  /* We assume that a font that supports 'A' supports ASCII chars.  */
+  entity = font_find_for_lface (f, attrs, spec, 'A');
   if (NILP (entity))
       /* No font is listed for SPEC, but each font-backend may have


   emacs -Q --eval "(push '(propertize \" \" 'face 'variable-pitch)

The attached PNGs show the difference. In both cases,

M-x describe-face variable-pitch <RET>

Face: variable-pitch (sample) (customize this face)
Documentation: The basic variable-pitch face.
Defined in `faces.el'.

        Family: Sans Serif
       Foundry: unspecified
         Width: unspecified
        Height: unspecified
        Weight: unspecified
         Slant: unspecified
    Foreground: unspecified
    Background: unspecified
     Underline: unspecified
      Overline: unspecified
Strike-through: unspecified
           Box: unspecified
       Inverse: unspecified
       Stipple: unspecified
          Font: unspecified
       Fontset: unspecified
       Inherit: unspecified


Attachment: good.png
Description: PNG image

Attachment: bad.png
Description: PNG image

reply via email to

[Prev in Thread] Current Thread [Next in Thread]