Re: Add M-x occur to the menu-bar

[Kim F. Storm]

Ted Zlatanov <address@hidden> writes:

> Here's another version of the next-error patch.  

Getting closer :-)

>                                                  What it does:
> 
> - move the next-error framework to simple.el, so it's always loaded.
>   This was next-error plus all the related functions.
> 
> - make the next-error-function always local (but not buffer-local)

The way you use make-local-variable doesn't accomplish that.

Either use make-variable-buffer-local at the global level (it will work
in this case, as simple.el is pre-loaded.

Or use

  (set (make-local-variable 'next-error-function) 'some-function)

everywhere you currently setq next-error-function.


>   (defun compilation-find-buffer (&optional other-buffer)
> !   (let ((next-error-buffer-p compilation-buffer-p))
> !     (next-error-find-buffer other-buffer)))

This does not function-bind next-error-buffer-p ...
Does it really work?

> + (make-local-variable 'next-error-function)

-- 
Kim F. Storm <address@hidden> http://www.cua.dk