bug#2994: Recent change to font.c:font_load_for_lface affects variable-pitch face in the modeline

bug#2994: Recent change to font.c:font_load_for_lface affects variable-pitch face in the modeline

[Juanma Barranquero]

[-- Attachment #1: Type: text/plain, Size: 1676 bytes --]

Package: emacs
X-Debbugs-CC: <handa@m17n.org>

This change affects using variable-pitch on the modeline:

--------------------------------------------------------------------------------
2009-04-13  Kenichi Handa  <handa@m17n.org>

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

--------------------------------------------------------------------------------
@@ -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
--------------------------------------------------------------------------------

Running

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

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

    Juanma

[-- Attachment #2: good.png --]
[-- Type: image/png, Size: 1191 bytes --]

[-- Attachment #3: bad.png --]
[-- Type: image/png, Size: 1287 bytes --]

bug#2994: Recent change to font.c:font_load_for_lface affects variable-pitch face in the modeline

[Miles Bader]

Juanma Barranquero <lekktu@gmail.com> writes:
> This change affects using variable-pitch on the modeline:

Are you sure that's wrong?  If the variable-pitch font you're using is
taller than the default font, wouldn't it make sense if it caused the
modeline to become taller too...?

[Obviously it depends on the particular font metrics]

-Miles

-- 
XML is like violence.  If it doesn't solve your problem, you're not
using enough of it.

bug#2994: Recent change to font.c:font_load_for_lface affects variable-pitch face in the modeline

[Juanma Barranquero]

On Tue, Apr 14, 2009 at 16:21, Miles Bader <miles@gnu.org> wrote:

> Are you sure that's wrong?

It is an unexpected change, and it look worse. If not a bug, hardly an
improvement.

> If the variable-pitch font you're using is
> taller than the default font, wouldn't it make sense if it caused the
> modeline to become taller too...?

Before that change, for (propertize " " 'face 'variable-pitch) Emacs
was selecting

    uniscribe:-outline-Arial-normal-normal-normal-sans-13-*-*-*-p-*-iso8859-1
(#x03)

Now, it selects

    uniscribe:-outline-FreeIdgSerif-normal-normal-normal-serif-13-*-*-*-p-*-iso10646-1
(#x02)

    Juanma

bug#2994: Recent change to font.c:font_load_for_lface affects variable-pitch face in the modeline

[Juanma Barranquero]

> On Tue, Apr 14, 2009 at 16:21, Miles Bader <miles@gnu.org> wrote:
>
>> Are you sure that's wrong?

All in all, it seems a bug.

After emacs -Q, I set the face for variable-pitch to DejaVu Sans:

  M-: (set-face-font 'variable-pitch "-outline-DejaVu
Sans-normal-normal-normal-sans-13-*-*-*-p-*-iso10646-1") <RET>

and describe-face agrees:

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

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

        Family: DejaVu Sans
       Foundry: outline
         Width: normal
        Height: 98
        Weight: normal
         Slant: normal
    Foreground: unspecified
    Background: unspecified
     Underline: unspecified
      Overline: unspecified
Strike-through: unspecified
           Box: unspecified
       Inverse: unspecified
       Stipple: unspecified
          Font: #<font-object -outline-DejaVu
Sans-normal-normal-normal-sans-13-*-*-*-p-*-iso10646-1>
       Fontset: unspecified
       Inherit: unspecified

However, I start a buffer in fundamental-mode, insert a variable-pitch space:

  C-x b test <RET>
  M-: (insert (propertize " " 'face 'variable-pitch)) <RET>

and it is in FreeIdgSerif, according to `describe-char':

        character: SPC (32, #o40, #x20)
preferred charset: ascii (ASCII (ISO646 IRV))
       code point: 0x20
           syntax:   	which means: whitespace
         category: .:Base, a:ASCII, l:Latin
      buffer code: #x20
        file code: #x20 (encoded by coding system iso-latin-1-dos)
          display: by this font (glyph code)
    uniscribe:-outline-FreeIdgSerif-normal-normal-normal-serif-13-*-*-*-p-*-iso10646-1
(#x02)

Character code properties: customize what to show
  name: SPACE
  general-category: Zs (Separator, Space)

There are text properties here:
  face                 variable-pitch


    Juanma

bug#2994: Recent change to font.c:font_load_for_lface affects variable-pitch face in the modeline

[Kenichi Handa]

In article <f7ccd24b0904140907r7fe80098qbb0e11e435208663@mail.gmail.com>, Juanma Barranquero <lekktu@gmail.com> writes:

> Before that change, for (propertize " " 'face 'variable-pitch) Emacs
> was selecting

>     uniscribe:-outline-Arial-normal-normal-normal-sans-13-*-*-*-p-*-iso8859-1
> (#x03)

> Now, it selects

>     uniscribe:-outline-FreeIdgSerif-normal-normal-normal-serif-13-*-*-*-p-*-iso10646-1
> (#x02)

As the family of your variable-pitch is "Sans Serif", if you
don't have a font of the exactly same family, what font is
actually selected for it depends on a font backend.  As my
change caused an extra font-sorting, perhaps it changed the
order of fonts that have the same score.

But, I installed a new change that avoid that extra
font-sorting.  Please try again.

---
Kenichi Handa
handa@m17n.org

bug#2994: Recent change to font.c:font_load_for_lface affects variable-pitch face in the modeline

[Juanma Barranquero]

On Thu, Apr 16, 2009 at 08:24, Kenichi Handa <handa@m17n.org> wrote:

> As the family of your variable-pitch is "Sans Serif", if you
> don't have a font of the exactly same family, what font is
> actually selected for it depends on a font backend.  As my
> change caused an extra font-sorting, perhaps it changed the
> order of fonts that have the same score.

There's something more going on, I think.

I'm setting variable-pitch to use a family of "DejaVu Serif":

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

        Family: DejaVu 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

and I *have* a DejaVu Serif face:

ELISP> (x-list-fonts "DejaVu Serif")
("-outline-DejaVu
Serif-normal-italic-normal-serif-*-*-*-*-p-*-iso10646-1"
"-outline-DejaVu
Serif-bold-italic-normal-serif-*-*-*-*-p-*-iso10646-1"
"-outline-DejaVu
Serif-bold-normal-normal-serif-*-*-*-*-p-*-iso10646-1"
"-outline-DejaVu
Serif-normal-normal-normal-serif-*-*-*-*-p-*-iso10646-1"
"-outline-DejaVu
Serif-normal-italic-normal-serif-*-*-*-*-p-*-iso8859-5"
"-outline-DejaVu
Serif-normal-italic-normal-serif-*-*-*-*-p-*-iso8859-2"
"-outline-DejaVu
Serif-normal-italic-normal-serif-*-*-*-*-p-*-iso8859-4"
"-outline-DejaVu
Serif-normal-italic-normal-serif-*-*-*-*-p-*-iso8859-9"
"-outline-DejaVu
Serif-normal-italic-normal-serif-*-*-*-*-p-*-iso8859-7"
"-outline-DejaVu
Serif-normal-italic-normal-serif-*-*-*-*-p-*-iso8859-1"
"-outline-DejaVu Serif-bold-italic-normal-serif-*-*-*-*-p-*-iso8859-5"
"-outline-DejaVu Serif-bold-italic-normal-serif-*-*-*-*-p-*-iso8859-2"
"-outline-DejaVu Serif-bold-italic-normal-serif-*-*-*-*-p-*-iso8859-4"
"-outline-DejaVu Serif-bold-italic-normal-serif-*-*-*-*-p-*-iso8859-9"
"-outline-DejaVu Serif-bold-italic-normal-serif-*-*-*-*-p-*-iso8859-7"
"-outline-DejaVu Serif-bold-italic-normal-serif-*-*-*-*-p-*-iso8859-1"
"-outline-DejaVu Serif-bold-normal-normal-serif-*-*-*-*-p-*-iso8859-5"
"-outline-DejaVu Serif-bold-normal-normal-serif-*-*-*-*-p-*-iso8859-2"
"-outline-DejaVu Serif-bold-normal-normal-serif-*-*-*-*-p-*-iso8859-4"
"-outline-DejaVu Serif-bold-normal-normal-serif-*-*-*-*-p-*-iso8859-9"
"-outline-DejaVu Serif-bold-normal-normal-serif-*-*-*-*-p-*-iso8859-7"
"-outline-DejaVu Serif-bold-normal-normal-serif-*-*-*-*-p-*-iso8859-1"
"-outline-DejaVu
Serif-normal-normal-normal-serif-*-*-*-*-p-*-iso8859-5"
"-outline-DejaVu
Serif-normal-normal-normal-serif-*-*-*-*-p-*-iso8859-2"
"-outline-DejaVu
Serif-normal-normal-normal-serif-*-*-*-*-p-*-iso8859-4"
"-outline-DejaVu
Serif-normal-normal-normal-serif-*-*-*-*-p-*-iso8859-9"
"-outline-DejaVu
Serif-normal-normal-normal-serif-*-*-*-*-p-*-iso8859-7"
"-outline-DejaVu
Serif-normal-normal-normal-serif-*-*-*-*-p-*-iso8859-1")

but still, when I insert (propertize " " 'face 'variable-pitch), I get
FreeIdgSerif:

        character: SPC (32, #o40, #x20)
preferred charset: ascii (ASCII (ISO646 IRV))
       code point: 0x20
           syntax:   	which means: whitespace
         category: .:Base, a:ASCII, l:Latin
      buffer code: #x20
        file code: #x20 (encoded by coding system utf-8-dos)
          display: by this font (glyph code)
    uniscribe:-outline-FreeIdgSerif-normal-normal-normal-serif-13-*-*-*-p-*-iso10646-1
(#x02)

Character code properties: customize what to show
  name: SPACE
  general-category: Zs (Separator, Space)
  canonical-combining-class: 0 (Spacing, split, enclosing, reordrant,
and Tibetan subjoined)
  bidi-class: WS (Whitespace)
  mirrored: N

There are text properties here:
  face                 variable-pitch

> But, I installed a new change that avoid that extra
> font-sorting.  Please try again.

It still fails, as shown above.

    Juanma

bug#2994: Recent change to font.c:font_load_for_lface affects variable-pitch face in the modeline

[Kenichi Handa]

In article <f7ccd24b0904160414h1482a99bid0afcccd7043d213@mail.gmail.com>, Juanma Barranquero <lekktu@gmail.com> writes:

> I'm setting variable-pitch to use a family of "DejaVu Serif":
[...]
> and I *have* a DejaVu Serif face:

ELISP> (x-list-fonts "DejaVu Serif")
> ("-outline-DejaVu Serif-normal-italic-normal-serif-*-*-*-*-p-*-iso10646-1"
[...]
> but still, when I insert (propertize " " 'face 'variable-pitch), I get
> FreeIdgSerif:

I can't reproduce your problem on Ubuntu.  Please try this:

% emacs -Q
ESC : (set-face-attribute 'variable-pitch nil :family "DejaVu Serif") RET
ESC : (setq font-log nil) RET
ESC : (insert (propertize " " 'face 'variable-pitch)) RET
M-x font-show-log RET

---
Kenichi Handa
handa@m17n.org

bug#2994: Recent change to font.c:font_load_for_lface affects variable-pitch face in the modeline

[Juanma Barranquero]

On Thu, Apr 16, 2009 at 14:14, Kenichi Handa <handa@m17n.org> wrote:

> % emacs -Q
> ESC : (set-face-attribute 'variable-pitch nil :family "DejaVu Serif") RET
> ESC : (setq font-log nil) RET
> ESC : (insert (propertize " " 'face 'variable-pitch)) RET
> M-x font-show-log RET

If I do exactly that, i.e., in the *scratch* buffer, the space is in
Courier New (the default font), and the font log says:

list: -outline-Courier New-normal-*-normal-mono-*-c-*-iso10646-1
  -outline-Courier New-normal-italic-normal-mono-*-c-*-iso10646-1
  -outline-Courier New-normal-italic-normal-mono-*-c-*-iso10646-1
  -outline-Courier New-normal-italic-normal-mono-*-c-*-iso10646-1
  -outline-Courier New-normal-italic-normal-mono-*-c-*-iso10646-1
  -outline-Courier New-normal-italic-normal-mono-*-c-*-iso10646-1
  -outline-Courier New-normal-italic-normal-mono-*-c-*-iso10646-1
  -outline-Courier New-normal-italic-normal-mono-*-c-*-iso10646-1
  -outline-Courier New-normal-italic-normal-mono-*-c-*-iso10646-1
  -outline-Courier New-normal-normal-normal-mono-*-c-*-iso10646-1
  -outline-Courier New-normal-normal-normal-mono-*-c-*-iso10646-1
  -outline-Courier New-normal-normal-normal-mono-*-c-*-iso10646-1
  -outline-Courier New-normal-normal-normal-mono-*-c-*-iso10646-1
  -outline-Courier New-normal-normal-normal-mono-*-c-*-iso10646-1
  -outline-Courier New-normal-normal-normal-mono-*-c-*-iso10646-1
  -outline-Courier New-normal-normal-normal-mono-*-c-*-iso10646-1
  -outline-Courier New-normal-normal-normal-mono-*-c-*-iso10646-1
  -outline-Courier New-normal-normal-normal-mono-*-c-*-iso10646-1
  -outline-Courier New-normal-italic-normal-mono-*-c-*-iso10646-1
  -outline-Courier New-normal-italic-normal-mono-*-c-*-iso10646-1
  ...
sort-by: -*-normal-italic-normal-*-13-*
uniscribe-match: -outline-Courier
New-normal-italic-normal-mono-13-*-c-*-iso10646-1
  uniscribe:-outline-Courier New-normal-italic-normal-mono-*-c-*-iso10646-1
match: -outline-Courier New-normal-italic-normal-mono-13-*-c-*-iso10646-1
  uniscribe:-outline-Courier New-normal-italic-normal-mono-*-c-*-iso10646-1
open: -outline-Courier New-normal-italic-normal-mono-*-c-*-iso10646-1
  uniscribe:-outline-Courier New-normal-italic-normal-mono-13-*-c-*-iso10646-1


If I do the same steps, but I switch to a new buffer "test" (in
fundamental mode) just before setting font-log, the space is in
FreeIdgSerif and the font log says:

uniscribe-list: -outline-DejaVu Serif-*-iso8859-1
  -outline-DejaVu Serif-normal-italic-normal-serif-*-p-*-iso8859-1
  -outline-DejaVu Serif-bold-italic-normal-serif-*-p-*-iso8859-1
  -outline-DejaVu Serif-bold-normal-normal-serif-*-p-*-iso8859-1
  -outline-DejaVu Serif-normal-normal-normal-serif-*-p-*-iso8859-1
w32font-list: -outline-DejaVu Serif-*-iso8859-1
  -outline-DejaVu Serif-normal-italic-normal-serif-*-p-*-iso8859-1
  -outline-DejaVu Serif-bold-italic-normal-serif-*-p-*-iso8859-1
  -outline-DejaVu Serif-bold-normal-normal-serif-*-p-*-iso8859-1
  -outline-DejaVu Serif-normal-normal-normal-serif-*-p-*-iso8859-1
list: -outline-DejaVu Serif-normal-normal-*-iso8859-1
  -outline-DejaVu Serif-normal-normal-normal-serif-*-p-*-iso8859-1
  -outline-DejaVu Serif-normal-normal-normal-serif-*-p-*-iso8859-1
sort-by: -*-normal-normal-normal-*-13-*
uniscribe-list: -outline-DejaVu Serif-*-ascii-0
w32font-list: -outline-DejaVu Serif-*-ascii-0
list: -outline-DejaVu Serif-normal-normal-*-ascii-0
uniscribe-list: -*-DejaVu Serif-*-iso8859-1
  -outline-DejaVu Serif-normal-italic-normal-serif-*-p-*-iso8859-1
  -outline-DejaVu Serif-bold-italic-normal-serif-*-p-*-iso8859-1
  -outline-DejaVu Serif-bold-normal-normal-serif-*-p-*-iso8859-1
  -outline-DejaVu Serif-normal-normal-normal-serif-*-p-*-iso8859-1
w32font-list: -*-DejaVu Serif-*-iso8859-1
  -outline-DejaVu Serif-normal-italic-normal-serif-*-p-*-iso8859-1
  -outline-DejaVu Serif-bold-italic-normal-serif-*-p-*-iso8859-1
  -outline-DejaVu Serif-bold-normal-normal-serif-*-p-*-iso8859-1
  -outline-DejaVu Serif-normal-normal-normal-serif-*-p-*-iso8859-1
list: -*-DejaVu Serif-normal-normal-*-iso8859-1
  -outline-DejaVu Serif-normal-normal-normal-serif-*-p-*-iso8859-1
  -outline-DejaVu Serif-normal-normal-normal-serif-*-p-*-iso8859-1
sort-by: -*-normal-normal-normal-*-13-*
uniscribe-list: -*-DejaVu Serif-*-ascii-0
w32font-list: -*-DejaVu Serif-*-ascii-0
list: -*-DejaVu Serif-normal-normal-*-ascii-0
uniscribe-list: -outline-*-iso8859-1
  -outline-FreeIdgSerif-normal-italic-normal-serif-*-p-*-iso8859-1
  -outline-FreeIdgSerif-bold-italic-normal-serif-*-p-*-iso8859-1
  -outline-FreeIdgSerif-bold-normal-normal-serif-*-p-*-iso8859-1
  -outline-FreeIdgSerif-normal-normal-normal-serif-*-p-*-iso8859-1
  -outline-Arial-normal-italic-normal-sans-*-p-*-iso8859-1
  -outline-Arial-bold-italic-normal-sans-*-p-*-iso8859-1
  -outline-Arial-bold-normal-normal-sans-*-p-*-iso8859-1
  -outline-Arial-normal-normal-normal-sans-*-p-*-iso8859-1
  -outline-Courier New-normal-italic-normal-mono-*-c-*-iso8859-1
  -outline-Courier New-bold-italic-normal-mono-*-c-*-iso8859-1
  -outline-Courier New-bold-normal-normal-mono-*-c-*-iso8859-1
  -outline-Courier New-normal-normal-normal-mono-*-c-*-iso8859-1
  -outline-Lucida Console-normal-normal-normal-mono-*-c-*-iso8859-1
  -outline-Lucida Sans Unicode-normal-normal-normal-sans-*-p-*-iso8859-1
  -outline-Times New Roman-normal-italic-normal-serif-*-p-*-iso8859-1
  -outline-Times New Roman-bold-italic-normal-serif-*-p-*-iso8859-1
  -outline-Times New Roman-bold-normal-normal-serif-*-p-*-iso8859-1
  -outline-Times New Roman-normal-normal-normal-serif-*-p-*-iso8859-1
  -outline-Verdana-bold-italic-normal-sans-*-p-*-iso8859-1
  ...
w32font-list: -outline-*-iso8859-1
  -raster-System-bold-normal-normal-sans-16-*-p-*-iso8859-1
  -raster-Fixedsys-normal-normal-normal-mono-15-*-c-*-iso8859-1
  -raster-Small Fonts-normal-normal-normal-sans-11-*-p-*-iso8859-1
  -raster-Small Fonts-normal-normal-normal-sans-10-*-p-*-iso8859-1
  -raster-Small Fonts-normal-normal-normal-sans-8-*-p-*-iso8859-1
  -raster-Small Fonts-normal-normal-normal-sans-6-*-p-*-iso8859-1
  -raster-Small Fonts-normal-normal-normal-sans-5-*-p-*-iso8859-1
  -raster-Small Fonts-normal-normal-normal-sans-3-*-p-*-iso8859-1
  -raster-MS Serif-normal-normal-normal-serif-35-*-p-*-iso8859-1
  -raster-MS Serif-normal-normal-normal-serif-27-*-p-*-iso8859-1
  -raster-MS Serif-normal-normal-normal-serif-21-*-p-*-iso8859-1
  -raster-MS Serif-normal-normal-normal-serif-19-*-p-*-iso8859-1
  -raster-MS Serif-normal-normal-normal-serif-16-*-p-*-iso8859-1
  -raster-MS Serif-normal-normal-normal-serif-13-*-p-*-iso8859-1
  -raster-MS Serif-normal-normal-normal-serif-11-*-p-*-iso8859-1
  -raster-MS Serif-normal-normal-normal-serif-10-*-p-*-iso8859-1
  -raster-Courier-normal-normal-normal-mono-20-*-c-*-iso8859-1
  -raster-Courier-normal-normal-normal-mono-16-*-c-*-iso8859-1
  -raster-Courier-normal-normal-normal-mono-13-*-c-*-iso8859-1
  ...
list: -outline-*-normal-normal-*-iso8859-1
  -outline-FreeIdgSerif-normal-normal-normal-serif-*-p-*-iso8859-1
  -outline-Arial-normal-normal-normal-sans-*-p-*-iso8859-1
  -outline-Courier New-normal-normal-normal-mono-*-c-*-iso8859-1
  -outline-Lucida Console-normal-normal-normal-mono-*-c-*-iso8859-1
  -outline-Lucida Sans Unicode-normal-normal-normal-sans-*-p-*-iso8859-1
  -outline-Times New Roman-normal-normal-normal-serif-*-p-*-iso8859-1
  -outline-Verdana-normal-normal-normal-sans-*-p-*-iso8859-1
  -outline-Comic Sans MS-normal-normal-normal-script-*-p-*-iso8859-1
  -outline-Impact-normal-normal-normal-sans-*-p-*-iso8859-1
  -outline-Georgia-normal-normal-normal-serif-*-p-*-iso8859-1
  -outline-Franklin Gothic Medium-normal-normal-normal-sans-*-p-*-iso8859-1
  -outline-Palatino Linotype-normal-normal-normal-serif-*-p-*-iso8859-1
  -outline-Tahoma-normal-normal-normal-sans-*-p-*-iso8859-1
  -outline-Tahoma-normal-normal-normal-sans-*-p-*-iso8859-1
  -outline-Trebuchet MS-normal-normal-normal-sans-*-p-*-iso8859-1
  -outline-Sylfaen-normal-normal-normal-serif-*-p-*-iso8859-1
  -outline-Microsoft Sans Serif-normal-normal-normal-sans-*-p-*-iso8859-1
  -outline-Arial Narrow-normal-normal-normal-sans-*-p-*-iso8859-1
  -outline-Rockwell-normal-normal-normal-serif-*-p-*-iso8859-1
  ...
sort-by: -*-normal-normal-normal-*-13-*
uniscribe-list: -outline-*-ascii-0
w32font-list: -outline-*-ascii-0
list: -outline-*-normal-normal-*-ascii-0
uniscribe-list: -*-iso8859-1
  -outline-FreeIdgSerif-normal-italic-normal-serif-*-p-*-iso8859-1
  -outline-FreeIdgSerif-bold-italic-normal-serif-*-p-*-iso8859-1
  -outline-FreeIdgSerif-bold-normal-normal-serif-*-p-*-iso8859-1
  -outline-FreeIdgSerif-normal-normal-normal-serif-*-p-*-iso8859-1
  -outline-Arial-normal-italic-normal-sans-*-p-*-iso8859-1
  -outline-Arial-bold-italic-normal-sans-*-p-*-iso8859-1
  -outline-Arial-bold-normal-normal-sans-*-p-*-iso8859-1
  -outline-Arial-normal-normal-normal-sans-*-p-*-iso8859-1
  -outline-Courier New-normal-italic-normal-mono-*-c-*-iso8859-1
  -outline-Courier New-bold-italic-normal-mono-*-c-*-iso8859-1
  -outline-Courier New-bold-normal-normal-mono-*-c-*-iso8859-1
  -outline-Courier New-normal-normal-normal-mono-*-c-*-iso8859-1
  -outline-Lucida Console-normal-normal-normal-mono-*-c-*-iso8859-1
  -outline-Lucida Sans Unicode-normal-normal-normal-sans-*-p-*-iso8859-1
  -outline-Times New Roman-normal-italic-normal-serif-*-p-*-iso8859-1
  -outline-Times New Roman-bold-italic-normal-serif-*-p-*-iso8859-1
  -outline-Times New Roman-bold-normal-normal-serif-*-p-*-iso8859-1
  -outline-Times New Roman-normal-normal-normal-serif-*-p-*-iso8859-1
  -outline-Verdana-bold-italic-normal-sans-*-p-*-iso8859-1
  ...
w32font-list: -*-iso8859-1
  -raster-System-bold-normal-normal-sans-16-*-p-*-iso8859-1
  -raster-Fixedsys-normal-normal-normal-mono-15-*-c-*-iso8859-1
  -raster-Small Fonts-normal-normal-normal-sans-11-*-p-*-iso8859-1
  -raster-Small Fonts-normal-normal-normal-sans-10-*-p-*-iso8859-1
  -raster-Small Fonts-normal-normal-normal-sans-8-*-p-*-iso8859-1
  -raster-Small Fonts-normal-normal-normal-sans-6-*-p-*-iso8859-1
  -raster-Small Fonts-normal-normal-normal-sans-5-*-p-*-iso8859-1
  -raster-Small Fonts-normal-normal-normal-sans-3-*-p-*-iso8859-1
  -raster-MS Serif-normal-normal-normal-serif-35-*-p-*-iso8859-1
  -raster-MS Serif-normal-normal-normal-serif-27-*-p-*-iso8859-1
  -raster-MS Serif-normal-normal-normal-serif-21-*-p-*-iso8859-1
  -raster-MS Serif-normal-normal-normal-serif-19-*-p-*-iso8859-1
  -raster-MS Serif-normal-normal-normal-serif-16-*-p-*-iso8859-1
  -raster-MS Serif-normal-normal-normal-serif-13-*-p-*-iso8859-1
  -raster-MS Serif-normal-normal-normal-serif-11-*-p-*-iso8859-1
  -raster-MS Serif-normal-normal-normal-serif-10-*-p-*-iso8859-1
  -raster-Courier-normal-normal-normal-mono-20-*-c-*-iso8859-1
  -raster-Courier-normal-normal-normal-mono-16-*-c-*-iso8859-1
  -raster-Courier-normal-normal-normal-mono-13-*-c-*-iso8859-1
  ...
list: -*-normal-normal-*-iso8859-1
  -outline-FreeIdgSerif-normal-normal-normal-serif-*-p-*-iso8859-1
  -outline-Arial-normal-normal-normal-sans-*-p-*-iso8859-1
  -outline-Courier New-normal-normal-normal-mono-*-c-*-iso8859-1
  -outline-Lucida Console-normal-normal-normal-mono-*-c-*-iso8859-1
  -outline-Lucida Sans Unicode-normal-normal-normal-sans-*-p-*-iso8859-1
  -outline-Times New Roman-normal-normal-normal-serif-*-p-*-iso8859-1
  -outline-Verdana-normal-normal-normal-sans-*-p-*-iso8859-1
  -outline-Comic Sans MS-normal-normal-normal-script-*-p-*-iso8859-1
  -outline-Impact-normal-normal-normal-sans-*-p-*-iso8859-1
  -outline-Georgia-normal-normal-normal-serif-*-p-*-iso8859-1
  -outline-Franklin Gothic Medium-normal-normal-normal-sans-*-p-*-iso8859-1
  -outline-Palatino Linotype-normal-normal-normal-serif-*-p-*-iso8859-1
  -outline-Tahoma-normal-normal-normal-sans-*-p-*-iso8859-1
  -outline-Tahoma-normal-normal-normal-sans-*-p-*-iso8859-1
  -outline-Trebuchet MS-normal-normal-normal-sans-*-p-*-iso8859-1
  -outline-Sylfaen-normal-normal-normal-serif-*-p-*-iso8859-1
  -outline-Microsoft Sans Serif-normal-normal-normal-sans-*-p-*-iso8859-1
  -outline-Arial Narrow-normal-normal-normal-sans-*-p-*-iso8859-1
  -outline-Rockwell-normal-normal-normal-serif-*-p-*-iso8859-1
  ...
sort-by: -*-normal-normal-normal-*-13-*
uniscribe-list: -*-ascii-0
w32font-list: -*-ascii-0
list: -*-normal-normal-*-ascii-0
uniscribe-match: -*-normal-normal-normal-*
  uniscribe:-outline-FreeIdgSerif-normal-normal-normal-serif-*-p-*-iso10646-1
match: -*-normal-normal-normal-*
  uniscribe:-outline-FreeIdgSerif-normal-normal-normal-serif-*-p-*-iso10646-1
open: -outline-FreeIdgSerif-normal-normal-normal-serif-*-p-*-iso10646-1
  uniscribe:-outline-FreeIdgSerif-normal-normal-normal-serif-13-*-p-*-iso10646-1
list: -outline-Courier New-normal-*-normal-mono-*-c-*-iso10646-1
  -outline-Courier New-normal-italic-normal-mono-*-c-*-iso10646-1
  -outline-Courier New-normal-italic-normal-mono-*-c-*-iso10646-1
  -outline-Courier New-normal-italic-normal-mono-*-c-*-iso10646-1
  -outline-Courier New-normal-italic-normal-mono-*-c-*-iso10646-1
  -outline-Courier New-normal-italic-normal-mono-*-c-*-iso10646-1
  -outline-Courier New-normal-italic-normal-mono-*-c-*-iso10646-1
  -outline-Courier New-normal-italic-normal-mono-*-c-*-iso10646-1
  -outline-Courier New-normal-italic-normal-mono-*-c-*-iso10646-1
  -outline-Courier New-normal-normal-normal-mono-*-c-*-iso10646-1
  -outline-Courier New-normal-normal-normal-mono-*-c-*-iso10646-1
  -outline-Courier New-normal-normal-normal-mono-*-c-*-iso10646-1
  -outline-Courier New-normal-normal-normal-mono-*-c-*-iso10646-1
  -outline-Courier New-normal-normal-normal-mono-*-c-*-iso10646-1
  -outline-Courier New-normal-normal-normal-mono-*-c-*-iso10646-1
  -outline-Courier New-normal-normal-normal-mono-*-c-*-iso10646-1
  -outline-Courier New-normal-normal-normal-mono-*-c-*-iso10646-1
  -outline-Courier New-normal-normal-normal-mono-*-c-*-iso10646-1
  -outline-Courier New-normal-italic-normal-mono-*-c-*-iso10646-1
  -outline-Courier New-normal-italic-normal-mono-*-c-*-iso10646-1
  ...
sort-by: -*-normal-italic-normal-*-13-*
uniscribe-match: -outline-Courier
New-normal-italic-normal-mono-13-*-c-*-iso10646-1
  uniscribe:-outline-Courier New-normal-italic-normal-mono-*-c-*-iso10646-1
match: -outline-Courier New-normal-italic-normal-mono-13-*-c-*-iso10646-1
  uniscribe:-outline-Courier New-normal-italic-normal-mono-*-c-*-iso10646-1
open: -outline-Courier New-normal-italic-normal-mono-*-c-*-iso10646-1
  uniscribe:-outline-Courier New-normal-italic-normal-mono-13-*-c-*-iso10646-1


Hope this helps,

    Juanma

bug#2994: Recent change to font.c:font_load_for_lface affects variable-pitch face in the modeline

[Kenichi Handa]

In article <f7ccd24b0904160705r61eb5d7ej94d7e4155d946384@mail.gmail.com>, Juanma Barranquero <lekktu@gmail.com> writes:

> If I do the same steps, but I switch to a new buffer "test" (in
> fundamental mode) just before setting font-log, the space is in
> FreeIdgSerif and the font log says:

> uniscribe-list: -outline-DejaVu Serif-*-iso8859-1
>   -outline-DejaVu Serif-normal-italic-normal-serif-*-p-*-iso8859-1
>   -outline-DejaVu Serif-bold-italic-normal-serif-*-p-*-iso8859-1
>   -outline-DejaVu Serif-bold-normal-normal-serif-*-p-*-iso8859-1
>   -outline-DejaVu Serif-normal-normal-normal-serif-*-p-*-iso8859-1
> w32font-list: -outline-DejaVu Serif-*-iso8859-1
>   -outline-DejaVu Serif-normal-italic-normal-serif-*-p-*-iso8859-1
>   -outline-DejaVu Serif-bold-italic-normal-serif-*-p-*-iso8859-1
>   -outline-DejaVu Serif-bold-normal-normal-serif-*-p-*-iso8859-1
>   -outline-DejaVu Serif-normal-normal-normal-serif-*-p-*-iso8859-1
> list: -outline-DejaVu Serif-normal-normal-*-iso8859-1
>   -outline-DejaVu Serif-normal-normal-normal-serif-*-p-*-iso8859-1
>   -outline-DejaVu Serif-normal-normal-normal-serif-*-p-*-iso8859-1
> sort-by: -*-normal-normal-normal-*-13-*
> uniscribe-list: -outline-DejaVu Serif-*-ascii-0

Hmmm, something wrong here.  It seems that this log means
the listed font -outline-DejaVu
Serif-normal-normal-normal-serif-*-p-*-iso8859-1 doesn't
contain ASCII character.

I'm now building Emacs on Windows XP to check what is wrong.

---
Kenichi Handa
handa@m17n.org

bug#2994: Recent change to font.c:font_load_for_lface affects variable-pitch face in the modeline

[Kenichi Handa]

In article <f7ccd24b0904140457l1beae924j74b8b11e5f943eb7@mail.gmail.com>, Juanma Barranquero <lekktu@gmail.com> writes:

> This change affects using variable-pitch on the modeline:

> --------------------------------------------------------------------------------
> 2009-04-13  Kenichi Handa  <handa@m17n.org>

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

This change requires the support of has_char method in a
font-driver, but I noticed that it is is not supported on
Windows and Mac.  So, I cancelled that change.

That change was to avoid selecting a bitmap font of `ja' and
`ko' adstyle in Xft backend.  But as the backned returns the
correct `avgwidth' property for bitmap fonts now, I think
such a font won't be selected if a user specifies a
preferable `avgwidth' as this:

In article <20090407022803.GB5160@metasyntax.net>, Taylor Venable <taylor@metasyntax.net> writes:

[...]
> My .emacs file looks like this:

> (defvar taylor-misc-fixed-large "-misc-fixed-medium-r-normal--13-120-75-75-c-70-iso10646-1"
>   "Miscellaneous fixed Unicode font - larger version.")

Taylor, could you please try again with the latest code?

---
Kenichi Handa
handa@m17n.org

bug#2994: Recent change to font.c:font_load_for_lface affects variable-pitch face in the modeline

[Juanma Barranquero]

On Tue, Apr 21, 2009 at 09:40, Kenichi Handa <handa@m17n.org> wrote:

> This change requires the support of has_char method in a
> font-driver, but I noticed that it is is not supported on
> Windows and Mac.  So, I cancelled that change.

OK, thanks.

    Juanma

bug#2994: Recent change to font.c:font_load_for_lface affects variable-pitch face in the modeline

[Taylor Venable]

On Tue, Apr 21, 2009 at 04:40:24PM +0900, Kenichi Handa wrote:
> That change was to avoid selecting a bitmap font of `ja' and
> `ko' adstyle in Xft backend.  But as the backned returns the
> correct `avgwidth' property for bitmap fonts now, I think
> such a font won't be selected if a user specifies a
> preferable `avgwidth' as this:
> 
> In article <20090407022803.GB5160@metasyntax.net>, Taylor Venable <taylor@metasyntax.net> writes:
> 
> [...]
> > My .emacs file looks like this:
> 
> > (defvar taylor-misc-fixed-large "-misc-fixed-medium-r-normal--13-120-75-75-c-70-iso10646-1"
> >   "Miscellaneous fixed Unicode font - larger version.")
> 
> Taylor, could you please try again with the latest code?

I built from updated CVS about 12 hours ago and it is still working
just fine.

-- 
Taylor Christopher Venable
http://real.metasyntax.net:2357/