Hi,

I hadn't looked at it from a session basis. My train of thought was this
patch provides more consistent behaviour of select-pane in that it always
switches to the pane you select. I'm not sure of the use case of being able
to select a different pane outside the current window but not switch to it.
Do others find a use for this? For me if I am selecting a pane I always
want to move to it regardless if it exists in my current window or not.

Kind regards,
George


George Brown


On 19 June 2014 08:47, Nicholas Marriott <nicholas.marri...@gmail.com>
wrote:

> I'm not sure it does make more sense. select-window doesn't change the
> session, so why should select-pane change the window?
>
>
> On Thu, Jun 19, 2014 at 08:42:33AM +0100, George Brown wrote:
> >    Dear all,
> >    Thanks for this patch Thomas it works for me, I applied against what
> was
> >    currently in git to build and it behaves exactly as I would expect.
> I'd
> >    love to see this mainlined and think this behaviour makes far more
> sense
> >    than the current release.
> >    Many thanks,
> >    George
> >    George Brown
> >
> >    On 18 June 2014 16:43, Thomas Adam <[1]tho...@xteddy.org> wrote:
> >
> >      On 18 June 2014 16:09, George Brown <[2]321.geo...@gmail.com>
> wrote:
> >      > Dear all,
> >      >
> >      > I was attempting this today and realised the "tmux select-pane"
> only
> >      > switches to panes in the active window. It appear this was raised
> in
> >      the
> >      > following ticket but I have not been able to find the actual
> >      discussion or
> >      > mentioned patch.
> >      >
> >      > [3]http://sourceforge.net/p/tmux/tickets/98/
> >      >
> >      > I am running 1.9a and would love to test the patch if someone
> could
> >      point me
> >      > to it.
> >
> >      This was what I had originally:
> >
> >      diff --git a/cmd-select-pane.c b/cmd-select-pane.c
> >      index 1a1072d..f989182 100644
> >      --- a/cmd-select-pane.c
> >      +++ b/cmd-select-pane.c
> >      @@ -67,6 +67,7 @@ cmd_select_pane_exec(struct cmd *self, struct
> cmd_q
> >      *cmdq)
> >      ** ** ** ** struct args ** ** ** ** ** ** *args = self->args;
> >      ** ** ** ** struct winlink ** ** ** ** ***wl;
> >      ** ** ** ** struct window_pane ** ** ***wp;
> >      + ** ** ** struct session ** ** ** ** ***s;
> >
> >      ** ** ** ** if (self->entry == &cmd_last_pane_entry ||
> args_has(args,
> >      'l')) {
> >      ** ** ** ** ** ** ** ** wl = cmd_find_window(cmdq, args_get(args,
> 't'),
> >      NULL);
> >      @@ -86,7 +87,7 @@ cmd_select_pane_exec(struct cmd *self, struct
> cmd_q
> >      *cmdq)
> >      ** ** ** ** ** ** ** ** return (CMD_RETURN_NORMAL);
> >      ** ** ** ** }
> >
> >      - ** ** ** if ((wl = cmd_find_pane(cmdq, args_get(args, 't'), NULL,
> >      &wp)) == NULL)
> >      + ** ** ** if ((wl = cmd_find_pane(cmdq, args_get(args, 't'), &s,
> &wp))
> >      == NULL)
> >      ** ** ** ** ** ** ** ** return (CMD_RETURN_ERROR);
> >
> >      ** ** ** ** server_unzoom_window(wp->window);
> >      @@ -111,6 +112,8 @@ cmd_select_pane_exec(struct cmd *self, struct
> cmd_q
> >      *cmdq)
> >      ** ** ** ** window_set_active_pane(wl->window, wp);
> >      ** ** ** ** server_status_window(wl->window);
> >      ** ** ** ** server_redraw_window_borders(wl->window);
> >      + ** ** ** if (session_select(s, wl->idx) == 0)
> >      + ** ** ** ** ** ** ** server_redraw_session(s);
> >
> >      ** ** ** ** return (CMD_RETURN_NORMAL);
> >      **}
> >
> >      It should still work.
> >      -- Thomas Adam
> >
> > References
> >
> >    Visible links
> >    1. mailto:tho...@xteddy.org
> >    2. mailto:321.geo...@gmail.com
> >    3. http://sourceforge.net/p/tmux/tickets/98/
>
> >
> ------------------------------------------------------------------------------
> > HPCC Systems Open Source Big Data Platform from LexisNexis Risk Solutions
> > Find What Matters Most in Your Big Data with HPCC Systems
> > Open Source. Fast. Scalable. Simple. Ideal for Dirty Data.
> > Leverages Graph Analysis for Fast Processing & Easy Data Exploration
> > http://p.sf.net/sfu/hpccsystems
>
> > _______________________________________________
> > tmux-users mailing list
> > tmux-users@lists.sourceforge.net
> > https://lists.sourceforge.net/lists/listinfo/tmux-users
>
>
------------------------------------------------------------------------------
HPCC Systems Open Source Big Data Platform from LexisNexis Risk Solutions
Find What Matters Most in Your Big Data with HPCC Systems
Open Source. Fast. Scalable. Simple. Ideal for Dirty Data.
Leverages Graph Analysis for Fast Processing & Easy Data Exploration
http://p.sf.net/sfu/hpccsystems
_______________________________________________
tmux-users mailing list
tmux-users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/tmux-users

Reply via email to