[Top][All Lists]
[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