Re: [PATCH] Add command to refresh status bar

2011-09-28 Thread Nicholas Marriott
Hi Applied this to OpenBSD, it'll be in SF at some point soonish. Cheers On Wed, Sep 28, 2011 at 09:25:46PM +0200, Marco Beck wrote: > On Wed, Sep 28, 2011 at 07:42:03PM +0100, Nicholas Marriott wrote: > > > This looks good, but I think CLIENT_REDRAW (set by > > server_redraw_client()) will al

Re: [PATCH] Add command to refresh status bar

2011-09-28 Thread Marco Beck
On Wed, Sep 28, 2011 at 07:42:03PM +0100, Nicholas Marriott wrote: > This looks good, but I think CLIENT_REDRAW (set by > server_redraw_client()) will already redraw the status line so no need > to set CLIENT_STATUS as well. Yeah, this seems superfluous. > Although, are you sure -S should redraw

Re: [PATCH] Add command to refresh status bar

2011-09-28 Thread Nicholas Marriott
Hi This looks good, but I think CLIENT_REDRAW (set by server_redraw_client()) will already redraw the status line so no need to set CLIENT_STATUS as well. Although, are you sure -S should redraw the entire client or should it not just redraw just the status line? On Wed, Sep 28, 2011 at 08:30:

Re: [PATCH] Add command to refresh status bar

2011-09-28 Thread Marco Beck
Hi Nicholas, On Tue, Sep 27, 2011 at 10:50:39PM +0100, Nicholas Marriott wrote: > > Can you make this a flag (-S or -s or whatever) to refresh-client > instead of a new command? Of course, this is much better. Updated patch (against SVN trunk) is attached. Thanks, Marco # HG changeset patch # U