emacs-pretest-bug
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: new style Emacs compile


From: Stefan Monnier
Subject: Re: new style Emacs compile
Date: 03 Dec 2003 09:43:59 -0500
User-agent: Gnus/5.09 (Gnus v5.9.0) Emacs/21.3.50

> Of course when I remove the limitation that matchers are implicitly
> anchored at beginning-of-line, everybody will have to add a ^ in front, or
> they may be matching more than what they thought.  (But then the standard
> regexps are already quite hairy :-)

I was thinking of keeping the behavior as before (for compatibility)
and only require an explicit ^ in the case where a new format is used.

> I like the "Patch already applied" message.  This is definitely a plus to
> just jumping there as I had intended.  But to offer this, compile would
> need some general "Bug already fixed" artificial intelligence, where diff
> could hook in...

Small matter of programming.

> Diff-mode already goes into the internal data-structures of the old
> compile message parser.  So it'll have to be looked at anyways.  Any ideas
> on how to integrate diff-mode's intelligence with compile's plain
> jump-to logic?

I don't remember diff-mode.el fiddling with compile too much.
It basically does its own thing.  There's a bit of support for compile
but it's incomplete.  OTOH diff.el uses compile extensively.
What I was saying is that diff.el should be changed to use diff-mode.el
and leave compile alone.

The only thing really needed w.r.t compile is to get the standard
keybindings working (and even then, only when requested: several people
complained when C-x v = cause a subsequent C-x ` to jump to the next diff
rather than to the next error in their *compile* buffer).  So we just
need a way for diff-mode to hook itself somewhere inside compile so compile
does nothing more than call diff-mode functions.


        Stefan




reply via email to

[Prev in Thread] Current Thread [Next in Thread]