bug#8876: 24.0.50; making `menu-bar-update-buffers' use another frame

[Drew Adams <drew.adams@oracle.com>]

Sending again, since the bug was archived (apparently replying to
the thread sends your reply to /dev/null if a bug has been archived -
quite a feature!).

--------------------------------------------------------

> > Attached.  Changelog entry:
> Thanks, installed,

Well, that only took things so far.  At the time I did that, Martin
had defined `pop-to-buffer-other-frame' in his version of window.el,
which I was using as a fix for another bug (since fixed).  That code
never made it into window.el, however.

So I guess it is not so easy now to assign `pop-to-buffer-other-frame'
to `menu-bar-select-buffer-function', i.e., to make it pop to the
buffer in another frame.

This was Martin's definition, at the time:

(defsubst pop-to-buffer-other-frame
  (&optional buffer-or-name norecord label)
  "Pop to buffer specified by BUFFER-OR-NAME on another frame.
The selected frame will be used only if there's no other choice.
Optional arguments BUFFER-OR-NAME, NORECORD and LABEL are as for
`pop-to-buffer'."
  (interactive "BPop to buffer on another frame:\nP")
  (pop-to-buffer buffer-or-name 'other-frame norecord label))

(I would not use a defsubst.)

But that is not compatible with the latest `pop-to-buffer'.
And when I look at the doc for `pop-to-buffer' I keel over.

It tells me I can use an ACTION argument (as Martin did here,
with value `other-frame').  But it sends me off to the doc for
`display-buffer', whose description of parameter ACTION is
incomprehensible to me.  WAY too complicated.

Can you please tell me what (simple) ACTION argument to pass to
`pop-to-buffer' to get it to use another frame?  The doc does not
make it sound like simply `other-frame' will suffice. ;-)

This is the function I'm using so far, with `pop-up-frames':

(lambda (buffer &optional other-window norecord)
  (interactive "BPop to buffer on another frame:\nP")
  (let ((pop-up-frames  t))
    (pop-to-buffer buffer other-window norecord)))

Is there a different simple way to do this now?  Hope so.

Thx.