GNU bug report logs - #12792
24.2.50; shell-mode renders all its output in a yellow face

Previous Next

Package: emacs;

Reported by: Dima Kogan <dima <at> secretsauce.net>

Date: Sat, 3 Nov 2012 08:59:02 UTC

Severity: normal

Tags: patch

Found in version 24.2.50

Done: Paul Eggert <eggert <at> cs.ucla.edu>

Bug is archived. No further changes may be made.

Full log


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

From: Eli Zaretskii <eliz <at> gnu.org>
To: Wolfgang Jenkner <wjenkner <at> inode.at>
Cc: 12792 <at> debbugs.gnu.org, dima <at> secretsauce.net
Subject: Re: bug#12792: 24.2.50;
	shell-mode renders all its output in a yellow face
Date: Sat, 03 Nov 2012 18:32:51 +0200
> From: Wolfgang Jenkner <wjenkner <at> inode.at>
> Date: Sat, 03 Nov 2012 16:37:59 +0100
> Cc: 12792 <at> debbugs.gnu.org
> 
> In bug#11883 you give this example
> 
> echo -e "\033[31mtest\033[39m"
> 
> but ansi-color has never supported SGR parameter 39.  However, before my
> change (the latter commit above), any unknown parameter would have the
> same effect as parameter 0 (turn off all rendition aspects), while it is
> simply ignored now (well, except for an "Invalid face reference: nil"
> message), see `ansi-color-apply-sequence'.  Perhaps, this explains the
> problem.

SGR parameter 39 means go back to the default foreground color.  How
can ansi-color not support it and still be useful?  IIUC, what the old
code did was exactly the right thing (or at least close, since 0 means
turn off _all_ attributes, not just the foreground color), so removing
it without replacing it with something that restores the default
foreground would be a bug, IMO.




This bug report was last modified 12 years and 248 days ago.

Previous Next


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