GNU bug report logs - #62994
Support styled underlines on TTY frames

Previous Next

Full log

Message #208 received at 62994 <at> debbugs.gnu.org (full text, mbox):

From: Eli Zaretskii <eliz <at> gnu.org>
To: Po Lu <luangruo <at> yahoo.com>, mohkale <at> kisara.moe
Cc: jporterbugs <at> gmail.com, 62994 <at> debbugs.gnu.org
Subject: Re: bug#62994: [PATCH v7] Add support for colored and styled
 underlines on tty frames
Date: Tue, 23 Apr 2024 09:02:18 +0300

> From: Po Lu <luangruo <at> yahoo.com>
> Cc: Jim Porter <jporterbugs <at> gmail.com>,  mohkale <at> kisara.moe,
>   62994 <at> debbugs.gnu.org
> Date: Tue, 23 Apr 2024 08:53:20 +0800
> 
> Eli Zaretskii <eliz <at> gnu.org> writes:
> 
> >> Cc: 62994 <at> debbugs.gnu.org
> >> Date: Mon, 22 Apr 2024 08:44:15 +0800
> >> From:  Po Lu via "Bug reports for GNU Emacs,
> >>  the Swiss army knife of text editors" <bug-gnu-emacs <at> gnu.org>
> >> 
> >> > * src/xterm.c (x_draw_glyph_string): Updated to use renamed
> >> > FACE_UNDERLINE_SINGLE and FACE_UNDERLINE_WAVE face_underline_type
> >> > enumerations.
> >> 
> >> This change should be accompanied by like updates to
> >> *_draw_glyph_string
> >> and related functions in the remaining terminal backends.
> >
> > I believe Jim Porter wanted to work on that.  Jim, have you made any
> > progress in this matter?
> >
> > In any case, given that implementing this for GUI displays should not
> > be a problem, I don't see a reason to block this changeset until the
> > GUI parts are done.  Hopefully, Jim or someone else will get it ready
> > soon.
> 
> I wasn't asking that the new underline styles be ported to other
> backends, but that the change to account for renamed enumerator values
> be ported to the remainder of the *term.[cm] files.  Without which they
> will fail to compile, which goes without saying.

I will not install a changeset that breaks the build, rest assured.  I
always build Emacs after applying patches from others, so I'd know if
the build is broken.  I presume Mohsin builds Emacs with his changes,
so unless he only builds Emacs --without-x, I presume the changes do
build?  Or what am I missing?

Previous Next

GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.