> There are two different arrows related to compilation and grep:
 > one is displayed in the compilation buffer, and another indicates
 > the line found by `next-error' in the source buffer.
 >
 > The latter can be displayed in any buffer and so should be
 > added to overlay-arrow-variable-list to avoid conflicts.
 > The former can appear only in the compilation buffer and so it
 > is sufficient to make it a local variable.  To display the arrow
 > in the compilation buffer on text-only terminals and to not share
 > the variable overlay-arrow-string with other packages, it could be
 > made local too.

Yes. Sorry, I didn't read your patch properly.

 > It seems everything works with the following patch:

It looks good now. In compilation-sentinel you might like to add:

   ;; buffer killed
   (set-process-buffer proc nil)
+  (delq 'next-error-overlay-arrow-position overlay-arrow-variable-list)

although I don't know what the performance penalty is, if any, if you don't.


Nick


_______________________________________________
Emacs-pretest-bug mailing list
[email protected]
http://lists.gnu.org/mailman/listinfo/emacs-pretest-bug

Reply via email to