On Fri, Jun 21, 2013 at 08:10:17PM -0700, Daniel Ralston wrote:
> Hi all,
>
> Bug Summary:
> When renumber-windows is set, killing a bunch of windows created in another
> grouped session will make the server die unexpectedly.
>
> Repro Steps:
>
> - In .tmux.conf: *set -g renumber-windows on*
> - Open two terminals.
> - In one terminal, run *tmux, *and create a bunch of new windows ("C-b c")
> - In the other terminal, run *tmux new-session -t *<name of other tmux
> session, e.g. 0>
> - In the second tmux client/session, kill a bunch of windows.(Created in
> the first client/session) ("C-b x")
>
> After killing a few, the server should die, killing both clients and
> displaying [Lost server] in both terminals.
Does the attached help any? I'm afraid I've not a lot of time to look in to
this, and I've not tested the patch either. I'm just supposing it'll do the
right thing. ;P
-- Thomas Adam
--
"Deep in my heart I wish I was wrong. But deep in my heart I know I am
not." -- Morrissey ("Girl Least Likely To" -- off of Viva Hate.)
diff --git a/session.c b/session.c
index 74eb06a..9b933a2 100644
--- a/session.c
+++ b/session.c
@@ -625,4 +625,6 @@ session_renumber_windows(struct session *s)
/* Free the old winlinks (reducing window references too). */
RB_FOREACH_SAFE(wl, winlinks, &old_wins, wl1)
winlink_remove(&old_wins, wl);
+
+ session_group_synchronize_from(s);
}
------------------------------------------------------------------------------
This SF.net email is sponsored by Windows:
Build for Windows Store.
http://p.sf.net/sfu/windows-dev2dev
_______________________________________________
tmux-users mailing list
tmux-users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/tmux-users
