* bug#2994: Recent change to font.c:font_load_for_lface affects variable-pitch face in the modeline
[not found] <mailman.5312.1239711841.31690.bug-gnu-emacs@gnu.org>
@ 2009-04-14 14:21 ` Miles Bader
2009-04-14 16:07 ` Juanma Barranquero
0 siblings, 1 reply; 12+ messages in thread
From: Miles Bader @ 2009-04-14 14:21 UTC (permalink / raw)
To: Juanma Barranquero; +Cc: Emacs Bug Tracker, 2994
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.
^ permalink raw reply [flat|nested] 12+ messages in thread* bug#2994: Recent change to font.c:font_load_for_lface affects variable-pitch face in the modeline
2009-04-14 14:21 ` bug#2994: Recent change to font.c:font_load_for_lface affects variable-pitch face in the modeline Miles Bader
@ 2009-04-14 16:07 ` Juanma Barranquero
2009-04-14 16:40 ` Juanma Barranquero
2009-04-16 6:24 ` Kenichi Handa
0 siblings, 2 replies; 12+ messages in thread
From: Juanma Barranquero @ 2009-04-14 16:07 UTC (permalink / raw)
To: Miles Bader; +Cc: 2994
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
^ permalink raw reply [flat|nested] 12+ messages in thread* bug#2994: Recent change to font.c:font_load_for_lface affects variable-pitch face in the modeline
2009-04-14 16:07 ` Juanma Barranquero
@ 2009-04-14 16:40 ` Juanma Barranquero
2009-04-16 6:24 ` Kenichi Handa
1 sibling, 0 replies; 12+ messages in thread
From: Juanma Barranquero @ 2009-04-14 16:40 UTC (permalink / raw)
To: Miles Bader; +Cc: 2994
> 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
^ permalink raw reply [flat|nested] 12+ messages in thread* bug#2994: Recent change to font.c:font_load_for_lface affects variable-pitch face in the modeline
2009-04-14 16:07 ` Juanma Barranquero
2009-04-14 16:40 ` Juanma Barranquero
@ 2009-04-16 6:24 ` Kenichi Handa
2009-04-16 11:14 ` Juanma Barranquero
1 sibling, 1 reply; 12+ messages in thread
From: Kenichi Handa @ 2009-04-16 6:24 UTC (permalink / raw)
To: Juanma Barranquero, 2994; +Cc: 2994, miles
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
^ permalink raw reply [flat|nested] 12+ messages in thread
* bug#2994: Recent change to font.c:font_load_for_lface affects variable-pitch face in the modeline
2009-04-16 6:24 ` Kenichi Handa
@ 2009-04-16 11:14 ` Juanma Barranquero
2009-04-16 12:14 ` Kenichi Handa
0 siblings, 1 reply; 12+ messages in thread
From: Juanma Barranquero @ 2009-04-16 11:14 UTC (permalink / raw)
To: Kenichi Handa; +Cc: 2994, miles
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
^ permalink raw reply [flat|nested] 12+ messages in thread* bug#2994: Recent change to font.c:font_load_for_lface affects variable-pitch face in the modeline
2009-04-16 11:14 ` Juanma Barranquero
@ 2009-04-16 12:14 ` Kenichi Handa
2009-04-16 14:05 ` Juanma Barranquero
0 siblings, 1 reply; 12+ messages in thread
From: Kenichi Handa @ 2009-04-16 12:14 UTC (permalink / raw)
To: Juanma Barranquero; +Cc: 2994, miles
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
^ permalink raw reply [flat|nested] 12+ messages in thread* bug#2994: Recent change to font.c:font_load_for_lface affects variable-pitch face in the modeline
2009-04-16 12:14 ` Kenichi Handa
@ 2009-04-16 14:05 ` Juanma Barranquero
2009-04-20 11:21 ` Kenichi Handa
0 siblings, 1 reply; 12+ messages in thread
From: Juanma Barranquero @ 2009-04-16 14:05 UTC (permalink / raw)
To: Kenichi Handa; +Cc: 2994, miles
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
^ permalink raw reply [flat|nested] 12+ messages in thread* bug#2994: Recent change to font.c:font_load_for_lface affects variable-pitch face in the modeline
2009-04-16 14:05 ` Juanma Barranquero
@ 2009-04-20 11:21 ` Kenichi Handa
0 siblings, 0 replies; 12+ messages in thread
From: Kenichi Handa @ 2009-04-20 11:21 UTC (permalink / raw)
To: Juanma Barranquero; +Cc: 2994, miles
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
^ permalink raw reply [flat|nested] 12+ messages in thread
* bug#2994: Recent change to font.c:font_load_for_lface affects variable-pitch face in the modeline
@ 2009-04-14 11:57 Juanma Barranquero
2009-04-21 7:40 ` Kenichi Handa
0 siblings, 1 reply; 12+ messages in thread
From: Juanma Barranquero @ 2009-04-14 11:57 UTC (permalink / raw)
To: Emacs Bug Tracker
[-- 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 --]
^ permalink raw reply [flat|nested] 12+ messages in thread* bug#2994: Recent change to font.c:font_load_for_lface affects variable-pitch face in the modeline
2009-04-14 11:57 Juanma Barranquero
@ 2009-04-21 7:40 ` Kenichi Handa
2009-04-21 12:06 ` Juanma Barranquero
2009-04-22 12:10 ` Taylor Venable
0 siblings, 2 replies; 12+ messages in thread
From: Kenichi Handa @ 2009-04-21 7:40 UTC (permalink / raw)
To: Juanma Barranquero, 2994; +Cc: taylor
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
^ permalink raw reply [flat|nested] 12+ messages in thread
* bug#2994: Recent change to font.c:font_load_for_lface affects variable-pitch face in the modeline
2009-04-21 7:40 ` Kenichi Handa
@ 2009-04-21 12:06 ` Juanma Barranquero
2009-04-22 12:10 ` Taylor Venable
1 sibling, 0 replies; 12+ messages in thread
From: Juanma Barranquero @ 2009-04-21 12:06 UTC (permalink / raw)
To: Kenichi Handa; +Cc: 2994
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
^ permalink raw reply [flat|nested] 12+ messages in thread* bug#2994: Recent change to font.c:font_load_for_lface affects variable-pitch face in the modeline
2009-04-21 7:40 ` Kenichi Handa
2009-04-21 12:06 ` Juanma Barranquero
@ 2009-04-22 12:10 ` Taylor Venable
1 sibling, 0 replies; 12+ messages in thread
From: Taylor Venable @ 2009-04-22 12:10 UTC (permalink / raw)
To: Kenichi Handa; +Cc: Juanma Barranquero, 2994
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/
^ permalink raw reply [flat|nested] 12+ messages in thread
end of thread, other threads:[~2009-04-22 12:10 UTC | newest]
Thread overview: 12+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
[not found] <mailman.5312.1239711841.31690.bug-gnu-emacs@gnu.org>
2009-04-14 14:21 ` bug#2994: Recent change to font.c:font_load_for_lface affects variable-pitch face in the modeline Miles Bader
2009-04-14 16:07 ` Juanma Barranquero
2009-04-14 16:40 ` Juanma Barranquero
2009-04-16 6:24 ` Kenichi Handa
2009-04-16 11:14 ` Juanma Barranquero
2009-04-16 12:14 ` Kenichi Handa
2009-04-16 14:05 ` Juanma Barranquero
2009-04-20 11:21 ` Kenichi Handa
2009-04-14 11:57 Juanma Barranquero
2009-04-21 7:40 ` Kenichi Handa
2009-04-21 12:06 ` Juanma Barranquero
2009-04-22 12:10 ` Taylor Venable
Code repositories for project(s) associated with this public inbox
https://git.savannah.gnu.org/cgit/emacs.git
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).