I take it that it is the “-t session:window.pane” syntax you would prefer? I
will make a new patch with that change.
On 10 12, 2013, at 16:49 , Nicholas Marriott <nicholas.marri...@gmail.com>
wrote:
> Thanks but I don't like this way of doing it.
>
> I think this should work the same way as attach-session.
>
> Take a look at the mail I just sent in the thread subject "patch
> allowing attach to take and select a pane".
>
> Both attach and switchc should work the same way, probably the way
> suggested in that thread.
>
> The alternative is that -w or -p or some other flag indicates -t is a
> window or pane rather than a session, but this isn't as nice for the
> user.
>
>
> On Sun, Dec 08, 2013 at 11:25:57AM +0100, Johannes Jakobsson wrote:
>> Add [-w window] to switch-client to specify which window in the
>> target-session to switch to.
>>
>> //Johannes
>>
>>> From d8efa53ec1104db86f4fd75be5a59d2317e7b673 Mon Sep 17 00:00:00 2001
>> From: Johannes Jakobsson <johannes.jakobs...@gmail.com>
>> Date: Sat, 7 Dec 2013 16:57:01 +0100
>> Subject: [PATCH] '-w' option to switch-client
>>
>> Example: switch-client -t s1 -w w1
>> ---
>> cmd-switch-client.c | 22 ++++++++++++++++++++--
>> 1 file changed, 20 insertions(+), 2 deletions(-)
>>
>> diff --git a/cmd-switch-client.c b/cmd-switch-client.c
>> index d101c52..68045c8 100644
>> --- a/cmd-switch-client.c
>> +++ b/cmd-switch-client.c
>> @@ -32,8 +32,8 @@ enum cmd_retval cmd_switch_client_exec(struct cmd *,
>> struct cmd_q *);
>>
>> const struct cmd_entry cmd_switch_client_entry = {
>> "switch-client", "switchc",
>> - "lc:npt:r", 0, 0,
>> - "[-lnpr] [-c target-client] [-t target-session]",
>> + "lc:npt:rw:", 0, 0,
>> + "[-lnpr] [-c target-client] [-t target-session] [-w window]",
>> CMD_READONLY,
>> cmd_switch_client_key_binding,
>> cmd_switch_client_exec
>> @@ -62,6 +62,7 @@ cmd_switch_client_exec(struct cmd *self, struct cmd_q
>> *cmdq)
>> struct args *args = self->args;
>> struct client *c;
>> struct session *s;
>> + struct winlink *wl;
>>
>> if ((c = cmd_find_client(cmdq, args_get(args, 'c'), 0)) == NULL)
>> return (CMD_RETURN_ERROR);
>> @@ -99,6 +100,23 @@ cmd_switch_client_exec(struct cmd *self, struct cmd_q
>> *cmdq)
>> if (s == NULL)
>> return (CMD_RETURN_ERROR);
>>
>> + wl = NULL;
>> + if (args_has(args, 'w')) {
>> + /* Prepend the choosen session name in front of the
>> + * window name to make cmd_find_window work.
>> + */
>> + char *sw;
>> + const char *w_name = args_get(args, 'w');
>> + xasprintf(&sw, "%s:%s", s->name, w_name);
>> + wl = cmd_find_window(cmdq, sw, &s);
>> + free(sw);
>> + if (wl == NULL) {
>> + cmdq_error(cmdq, "can't find window");
>> + return (CMD_RETURN_ERROR);
>> + }
>> + session_set_current(s, wl);
>> + }
>> +
>> if (c->session != NULL)
>> c->last_session = c->session;
>> c->session = s;
>> --
>> 1.8.3.2
>>
>>
>> ------------------------------------------------------------------------------
>> Sponsored by Intel(R) XDK
>> Develop, test and display web and hybrid apps with a single code base.
>> Download it for free now!
>> http://pubads.g.doubleclick.net/gampad/clk?id=111408631&iu=/4140/ostg.clktrk
>> _______________________________________________
>> tmux-users mailing list
>> tmux-users@lists.sourceforge.net
>> https://lists.sourceforge.net/lists/listinfo/tmux-users
------------------------------------------------------------------------------
Rapidly troubleshoot problems before they affect your business. Most IT
organizations don't have a clear picture of how application performance
affects their revenue. With AppDynamics, you get 100% visibility into your
Java,.NET, & PHP application. Start your 15-day FREE TRIAL of AppDynamics Pro!
http://pubads.g.doubleclick.net/gampad/clk?id=84349831&iu=/4140/ostg.clktrk
_______________________________________________
tmux-users mailing list
tmux-users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/tmux-users
