23.0.60; Underlining text including U+00F6, U+00E4

23.0.60; Underlining text including U+00F6, U+00E4

[Markus Triska]

When I open the following file (enriched-mode) with "emacs -Q":

   http://www.logic.at/prolog/test.doc

it looks like this:

   http://www.logic.at/prolog/test.png

All characters up to, and including, U+00F6 are underlined, and the
following are not, although test.doc says all should be underlined.

This also occurs, for example, in message titles with Gnus (where the
current mail is underlined), if an author name contains U+00E4, U+00E9
and several others.

In GNU Emacs 23.0.60.2 (i386-apple-darwin8.11.1, GTK+ Version 2.12.9)
 of 2008-05-19 on mt-computer.local
Windowing system distributor `The XFree86 Project, Inc', version 11.0.40400000
Important settings:
  value of $LC_ALL: nil
  value of $LC_COLLATE: nil
  value of $LC_CTYPE: nil
  value of $LC_MESSAGES: nil
  value of $LC_MONETARY: nil
  value of $LC_NUMERIC: nil
  value of $LC_TIME: nil
  value of $LANG: nil
  value of $XMODIFIERS: nil
  locale-coding-system: nil
  default-enable-multibyte-characters: t

Re: 23.0.60; Underlining text including U+00F6, U+00E4

[Kenichi Handa]

In article <20080522125825.8F2F2889A35@mt-computer.local>, Markus Triska <markus.triska@gmx.at> writes:

> When I open the following file (enriched-mode) with "emacs -Q":

>    http://www.logic.at/prolog/test.doc

> it looks like this:

>    http://www.logic.at/prolog/test.png

> All characters up to, and including, U+00F6 are underlined, and the
> following are not, although test.doc says all should be underlined.

I've just installed a fix.

---
Kenichi Handa
handa@ni.aist.go.jp

Re: 23.0.60; Underlining text including U+00F6, U+00E4

[YAMAMOTO Mitsuharu]

>>>>> On Fri, 23 May 2008 13:45:32 +0900, Kenichi Handa <handa@m17n.org> said:

> In article <20080522125825.8F2F2889A35@mt-computer.local>, Markus
> Triska <markus.triska@gmx.at> writes:
>> When I open the following file (enriched-mode) with "emacs -Q":

>> http://www.logic.at/prolog/test.doc

>> it looks like this:

>> http://www.logic.at/prolog/test.png

>> All characters up to, and including, U+00F6 are underlined, and the
>> following are not, although test.doc says all should be underlined.

> I've just installed a fix.

Isn't `position' supposed to be relative to the baseline?  If so,
these lines in xterm.c seems to be wrong.

  2727		      if (x_underline_at_descent_line)
  2728			position = (s->height - thickness) - s->ybase;

  2748		  if (s->y + s->height <= s->ybase + position)
  2749		    position = s->y + s->height - 1;

				     YAMAMOTO Mitsuharu
				mituharu@math.s.chiba-u.ac.jp

Re: 23.0.60; Underlining text including U+00F6, U+00E4

[Kenichi Handa]

In article <wlve15nt8g.wl%mituharu@math.s.chiba-u.ac.jp>, YAMAMOTO Mitsuharu <mituharu@math.s.chiba-u.ac.jp> writes:

> Isn't `position' supposed to be relative to the baseline?  If so,
> these lines in xterm.c seems to be wrong.

>   2727		      if (x_underline_at_descent_line)
>   2728			position = (s->height - thickness) - s->ybase;

>   2748		  if (s->y + s->height <= s->ybase + position)
>   2749		    position = s->y + s->height - 1;

Oops, thank you for pointing out that.  You are right.  But,
as I'm now at home, and can't commit a fix.  I'll fix it
next week, or if you can, please go ahead.

---
Kenichi Handa
handa@ni.aist.go.jp

Re: 23.0.60; Underlining text including U+00F6, U+00E4

[Kenichi Handa]

In article <E1JzgkC-0005TM-3X@etlken.m17n.org>, Kenichi Handa <handa@m17n.org> writes:

> In article <wlve15nt8g.wl%mituharu@math.s.chiba-u.ac.jp>, YAMAMOTO Mitsuharu <mituharu@math.s.chiba-u.ac.jp> writes:
> > Isn't `position' supposed to be relative to the baseline?  If so,
> > these lines in xterm.c seems to be wrong.

> >   2727		      if (x_underline_at_descent_line)
> >   2728			position = (s->height - thickness) - s->ybase;

> >   2748		  if (s->y + s->height <= s->ybase + position)
> >   2749		    position = s->y + s->height - 1;

> Oops, thank you for pointing out that.  You are right.  But,
> as I'm now at home, and can't commit a fix.  I'll fix it
> next week, or if you can, please go ahead.

I've just installed a fix.

---
Kenichi Handa
handa@ni.aist.go.jp