GNU bug report logs - #32676
[PATCH] Add option to highlight the 'next-error' error message

Previous Next

Package: emacs;

Reported by: Ernesto Alfonso <erjoalgo <at> gmail.com>

Date: Mon, 10 Sep 2018 05:24:02 UTC

Severity: wishlist

Tags: fixed, patch

Fixed in version 28.1

Done: Lars Ingebrigtsen <larsi <at> gnus.org>

Bug is archived. No further changes may be made.

Full log

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

From: Juri Linkov <juri <at> linkov.net>
To: Ernesto Alfonso <erjoalgo <at> gmail.com>
Cc: Eli Zaretskii <eliz <at> gnu.org>, Robert Pluim <rpluim <at> gmail.com>,
 32676 <at> debbugs.gnu.org
Subject: Re: bug#32676: [PATCH] Add option to highlight the 'next-error' error
 message
Date: Mon, 08 Apr 2019 22:36:38 +0300

> I'd like to know if this patch is still being considered?

Why not?  Your patch provides a helpful feature.  I see only 2 problems
with its latest version:

1. compilation-current-error should be generalized not to be too
   compilation-specific;

2. next-error-hook should not be used for core features,
   you could call next-error-message-highlight directly
   from next-error-found.

PS: maybe a better name for defcustom would be next-error-message-highlight,
    not next-error-message-highlight-p, to be more future-proof,
    for the case when someone might want to add more choices later
    (e.g. fringe, timers, etc.)
This bug report was last modified 4 years and 251 days ago.

Previous Next

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