Re: [PATCH] Allow custom key tables for e.g. multiple keystroke?bindings.

2015-04-13 Thread Nicholas Marriott
Here's the latest diff. It actually seems to work fine, although I haven't done a lot of testing. IIRC I wasn't wild about the cmd-list-keys.c and cmd-bind-key.c changes; certainly lsk -T should error on an unknown table, same as bind -T. I think the manpage bits could do with some improvement to

Re: [PATCH] Allow custom key tables for e.g. multiple keystroke bindings.

2015-04-13 Thread Nicholas Marriott
I made a load of changes and sent them to the author but he didn't respond and I haven't had time to pick it up again. I'll send the latest to you when I get home if you want to continue with it. IIRC it was pretty much done just needed testing and maybe man page... Original message --

Re: [PATCH] Allow custom key tables for e.g. multiple keystroke bindings.

2015-04-13 Thread Ross Hadden
Keith Amling palantir.com> writes: > > > > > Or are you thinking of creating the table after running the set command? > > > > This shouldn't work - you shouldn't be able to set a client to a > > > > nonexistent table. > > > > > > I guess I don't super care what happens since I won't personally

Re: [PATCH] Update environment when switching sessions

2015-04-13 Thread Nicholas Marriott
probably, thanks. it looks ok but i'm not sure if it is a good idea On Mon, Apr 13, 2015 at 09:13:29AM +0100, Si Beaumont wrote: > On Wed, 01 Apr 2015 at 11:00, Si Beaumont wrote: > > On Wed, 01 Apr 2015 at 01:24, Thomas Adam wrote: > > > On Tue, Mar 31, 2015 at 08:56:15AM +0100, Si Beaumont wrot

Re: [PATCH] Update environment when switching sessions

2015-04-13 Thread Si Beaumont
On Wed, 01 Apr 2015 at 11:00, Si Beaumont wrote: > On Wed, 01 Apr 2015 at 01:24, Thomas Adam wrote: > > On Tue, Mar 31, 2015 at 08:56:15AM +0100, Si Beaumont wrote: > > > + update = options_get_string(&global_s_options, > > > "update-environment"); > > > > I think you want &s->opt