Nit: + configuration_mode_up, + configuration_mode_down
We use C99; it'd be nice if you added a trailing ‘,’ here so that future additions don't also include a superfluous deletion.
Also nit: - print_current_configuration(ctx->connection); + case XKB_KEY_p:print_current_configuration();
Missing newline between case XKB_KEY_p: and print_current_configuration()?
I find the use of the conf global strange, but actual use looks fine. The current code does no error-checking, so we're not losing anything.
Actually... this will crash if you press ‘p’ to print the display configuration before you do anything.
Maybe this should be factored into a little accessor function? Either that or print_current_configuration() needs a null-guard.
« Back to merge proposal
Nit: mode_up, mode_down
+ configuration_
+ configuration_
We use C99; it'd be nice if you added a trailing ‘,’ here so that future additions don't also include a superfluous deletion.
Also nit: configuration( ctx->connection ); p:print_ current_ configuration( );
- print_current_
+ case XKB_KEY_
Missing newline between case XKB_KEY_p: and print_current_ configuration( )?
I find the use of the conf global strange, but actual use looks fine. The current code does no error-checking, so we're not losing anything.
Actually... this will crash if you press ‘p’ to print the display configuration before you do anything.
Maybe this should be factored into a little accessor function? Either that or print_current_ configuration( ) needs a null-guard.