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
