GNU bug report logs - #60580
29.0.60; crash in xdisp.c with modus theme and display-fill-column-indicator

Previous Next

Full log

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

From: Protesilaos Stavrou <info <at> protesilaos.com>
To: Eli Zaretskii <eliz <at> gnu.org>, yandros <at> gmail.com
Cc: 60580 <at> debbugs.gnu.org
Subject: Re: bug#60580: 29.0.60; crash in xdisp.c with modus theme and
 display-fill-column-indicator
Date: Fri, 06 Jan 2023 06:01:05 +0200

> From: Eli Zaretskii <eliz <at> gnu.org>
> Date: Thu,  5 Jan 2023 22:03:48 +0200
>
>> Cc: 60580 <at> debbugs.gnu.org
>> Date: Thu, 05 Jan 2023 21:57:39 +0200
>> From: Eli Zaretskii <eliz <at> gnu.org>
>> 
>> > From: chad <yandros <at> gmail.com>
>> > Date: Thu, 5 Jan 2023 14:19:48 -0500
>> > 
>> > From emacs -q, evaluate:
>> > 
>> >   (custom-set-variables '(custom-enabled-themes '(modus-vivendi)))
>> >   (global-display-fill-column-indicator-mode 1)
>> > 
>> > Emacs will fail the eassert in append_stretch_glyph at
>> > xdisp.c:30993.
>> 
>> Doesn't happen here.  What is your default face's font with that theme?
>
> I guess the culprit is this bit:
>
>   ;;;;; display-fill-column-indicator-mode
>       `(fill-column-indicator ((,class :height 1 :background ,bg-region :foreground ,bg-region)))
>
> I have no idea what is ":height 1" about, but I think it's wrong and
> should be replaced with some reasonable value.

The ":height 1" has been there for many versions and never caused a
problem of this sort.  If this is the cause, I will change it right
away.  The reason it is like this is to make the indicator a contiguous
line that is not affected by variations in font heights and/or a high
'line-spacing' value.  Otherwise the indicator can appear as a dashed
line.

-- 
Protesilaos Stavrou
https://protesilaos.com

Previous Next

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