Re: if-shell background changes

2013-03-03 Thread Nicholas Marriott
This is fine for if-shell but wrong for run-shell because wp can't be NULL, it is deferenced a bit later in the function. I suggest making wp_id signed and setting it to -1 if the output should be discarded in cmd_run_shell_print. On Wed, Feb 27, 2013 at 11:28:01AM -0500, Ben Boeckel wrote: > On

Re: if-shell background changes

2013-02-27 Thread Ben Boeckel
On Tue, Feb 26, 2013 at 19:55:39 +, Thomas Adam wrote: > This is, I presume, only when the server is initially started, yes? That > makes sense because here we see in cmd-if-shell.c:cmd_if_shell_exec(): Yes. Yep, that works. Patch attached which applied (I'm guessing the inlining mangled s

Re: if-shell background changes

2013-02-26 Thread Thomas Adam
Hi, On Tue, Feb 26, 2013 at 06:37:19PM +, Ben Boeckel wrote: > Hi, > > I updated to the newest master recently and it seems as though the > synchronous if-shell support has broken usage of it from ~/.tmux.conf. > At the end of my .tmux.conf I have the following: > > # 256-color support > if-