Re: Patch - add move-pane command, add -b argument to join-pane

2012-02-06 Thread Nicholas Marriott
Hi Looks good thanks, but move-pane will need to go in the man page? On Sat, Feb 04, 2012 at 12:25:40PM -0800, George Nachman wrote: > This patch adds a new command, move-pane. It is a wafer-thin wrapper > around the implementation of join-pane and simply removes the > restriction that source a

Re: Permission error when getting cwd

2012-02-06 Thread Nicholas Marriott
Hi I'm pretty sure that should be "root != NULL" ;-). On Sun, Feb 05, 2012 at 11:49:39PM -0500, Ben Boeckel wrote: > If you have a process running in a window and create a new window with > it focused, a NULL dereference can occur if the process is not owned by > the user (e.g., a root shell).

Re: Permission error when getting cwd

2012-02-06 Thread Ben Boeckel
On Mon, Feb 06, 2012 at 09:16:47 +, Nicholas Marriott wrote: > I'm pretty sure that should be "root != NULL" ;-). Oops, yes. I had originally had it down after the complete_path label and returning (s->cwd) itself, but falling through seems like the better option. Guess I didn't swap the compa

Re: Patch - add -F to display-message command

2012-02-06 Thread Ben Boeckel
-BEGIN PGP SIGNED MESSAGE- Hash: SHA512 On Mon, Feb 06, 2012 at 06:06:04 GMT, George Nachman wrote: > As we discussed earlier, this patch adds -F format to display-message. > If -F is passed then that format is used in preference to the > 'message' argument. I have this independently here