Skip to content

Refactor events to commands

Olivier Girondel requested to merge refactor_events into master

This binds all keyboard controls to one command (instead of a triplet of To/Command/Argument).

Makes code cleaner, this will be used to simplify the MIDI interface POC.

Next step will be to save/load the banks data in JSON.

@sub26nico:

  • Documentation for the banks is in the wiki
  • Test all shortcuts work as expected
  • Test the banks feature
  • Check the generated files (man page, manual) are ok
  • Sequences previous/next now should be in the right order (decreasing/increasing ids)

@jko42:

  • All commits starting with "Add SOME_COMMAND" are basically backports of the old system, I have checked them and it looks ok
  • a35826a0 fixes some banks management issues
  • Can be tested with -DDEBUG_COMMANDS

Finally

  • Feel free to submit ideas for modifications in #84 (closed) (we will do that after this branch is merged)
Edited by Olivier Girondel

Merge request reports