Yes, that works for me.  My workaround was to save the current-buffer after occur-mode-goto-occurrence, and then set-buffer outside the with-current-buffer, but apparently that was overkill.

On Thu, Aug 17, 2017 at 4:36 AM, Nathan Moreau <nathan.moreau@m4x.org> wrote:

Here  is a patch that fixes the reported problem.

A minimal reproduction test:

(ert-deftest totoro ()
  (set-buffer "*scratch*")
  (insert "setq")
  (setq next-error-recenter '(4))
  (occur "setq")
  (next-error))

It seems that occur-mode-goto-occurrence should be called after
restoring the original buffer.

Nathan

--

Thanks,

Jonathan Kotta

Hofstadter's Law:
    It always takes longer than you expect, even
    when you take into account Hofstadter's Law.