Re: appointment display during isearch replaces buffer contents with history-element

[martin rudalics]

>   (if (equal (selected-window) (minibuffer-window))
>       (if (other-window 1)
>           (select-window (other-window 1))
>         (if (display-multi-frame-p)
>             (select-frame (other-frame 1)))))
>
> This code makes no sense. other-window and other-frame select a window
> or frame--they don't return it. In fact, the return value is always nil,
> so this can't possibly be right.

Does it work if you use

  (or (not (minibufferp))
      (and (display-multi-frame-p)
           (progn
             (other-frame 1)
             (not (minibufferp))))
      (other-window 1))

instead?