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

2014-05-27 Thread Nicholas Marriott
Hi Sorry for the delay. Here is an updated version of the diff with some changes, mostly style changes but a few bigger: - Added -T to lsk. It should possibly grow -n as well. - I don't see a need for a for (;;) loop and break/continue in server_client_handle_key - it looks like we can only l

[PATCH 3d/3] Allow custom key tables for e.g. multiple keystroke bindings.

2014-05-15 Thread Keith Amling
> > + format_add(ft, "client_keytablename", "%s", c->keytable->name); > > Let's call it client_key_table to fit the style of most of the others. Done. > > +struct key_table * > > +key_bindings_lookup_table(const char *name, int create) > > I don't think it is intuitive for a lookup function t