Thanks for the report. I think it is better to move the -k section of
code after the -c section instead so I've done that.
On Fri, Nov 22, 2013 at 04:44:09PM +0000, Jo?o Crist?v?o wrote:
> Ok, it seems a simple:
>
> if (s->curw != NULL) {
> format_winlink(ft, s, s->curw);
> format_window_pane(ft, s->curw->window->active);
> }
>
> In line 115 should be enough... don't apply the window keys if no key
> is available. The rest works as expected.
>
> Cheers
>
> 2013/11/22 Jo?o Crist?v?o <jmacristo...@gmail.com>:
> > Hi,
> >
> > I've been using tmux for some time now, I'm a big fan!
> >
> > Today, however, I think I stumbled upon a bug, easy to reproduce:
> >
> > tmux new-session -Ad -s dev
> > tmux new-window -k -c /tmp -t dev:1
> > tmux attach-session -t dev
> >
> > It reports "no sessions", as the server has crashed.
> >
> > The problem lies with using both the -k and the -c options. When used
> > alone, it does not manifest itself.
> >
> > After a very quick glance in the code (git latest), I would say the
> > problem lies in cmd_new-window.c:
> >
> > 88: if (wl != NULL && args_has(args, 'k')) {
> > (...)
> > 101: s->curw = NULL;
> >
> > 110: if (args_has(args, 'c')) {
> > (...)
> > 115: format_winlink(ft, s, s->curw);
> >
> > But I might be wrong.
> >
> > Cheers,
> > Jo?o
>
> ------------------------------------------------------------------------------
> Shape the Mobile Experience: Free Subscription
> Software experts and developers: Be at the forefront of tech innovation.
> Intel(R) Software Adrenaline delivers strategic insight and game-changing
> conversations that shape the rapidly evolving mobile landscape. Sign up now.
> http://pubads.g.doubleclick.net/gampad/clk?id=63431311&iu=/4140/ostg.clktrk
> _______________________________________________
> tmux-users mailing list
> tmux-users@lists.sourceforge.net
> https://lists.sourceforge.net/lists/listinfo/tmux-users
------------------------------------------------------------------------------
Shape the Mobile Experience: Free Subscription
Software experts and developers: Be at the forefront of tech innovation.
Intel(R) Software Adrenaline delivers strategic insight and game-changing
conversations that shape the rapidly evolving mobile landscape. Sign up now.
http://pubads.g.doubleclick.net/gampad/clk?id=63431311&iu=/4140/ostg.clktrk
_______________________________________________
tmux-users mailing list
tmux-users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/tmux-users
