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
> > + 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