Subject: Re: bug#70819: [PATCH] Make clone-indirect-buffer-other-window use other window

From: Eli Zaretskii <[email-masked]>
Date: Sat, 08 Jun 2024 14:45:57 +0300

Ping! Ping! Richard, please augment your patch according to Martin's
comments, so we could install this.

> Cc: martin rudalics <[email-masked]>, [email-masked],
>  [email-masked]
> Date: Sat, 25 May 2024 10:49:56 +0300
> From: Eli Zaretskii <[email-masked]>
> 
> Ping!  Richard, could you please augment your pat6ch according to
> Martin's comments and resubmit?
> 
> > Date: Mon, 13 May 2024 10:05:52 +0200
> > Cc: Eli Zaretskii <[email-masked]>, [email-masked]
> > From: martin rudalics <[email-masked]>
> > 
> >  > -(defun clone-indirect-buffer-other-window (newname display-flag &optional norecord)
> >  > +(defun clone-indirect-buffer-other-window (newname &optional norecord)
> >  >     "Like `clone-indirect-buffer' but display in another window."
> >  >     (interactive
> >  >      (progn
> >  > @@ -10658,8 +10658,8 @@ Returns the newly created indirect buffer."
> >  >        (list (if current-prefix-arg
> >  >   	       (read-buffer "Name of indirect buffer: " (current-buffer)))
> >  >   	   t)))
> >  > -  (let ((pop-up-windows t))
> >  > -    (clone-indirect-buffer newname display-flag norecord)))
> >  > +  (switch-to-buffer-other-window
> >  > +   (clone-indirect-buffer newname nil norecord)))
> > 
> > I see two problems:
> > 
> > (1) 'clone-indirect-buffer-other-window' would still have to obey the
> >      DISPLAY_FLAG argument.  Some application may rely on the fact that
> >      'clone-indirect-buffer-other-window' doesn't display the buffer if
> >      it is nil.
> > 
> > (2) 'switch-to-buffer-other-window' does not get passed the NORECORD
> >      argument.
> > 
> > martin
> > 
> 
> 
> 
>