bjoerngiesler / BBDroids

Code for the Bavarian R2 Builders' Droid Control System, consisting of dual miniature multi-axis remote controls and a droid control board, currently for BB-8. Please refer to the Wiki for reference.
Apache License 2.0
6 stars 3 forks source link

Move most console commands into bb::Subsystem #19

Closed bjoerngiesler closed 1 year ago

bjoerngiesler commented 1 year ago

Command concept is not really clean.

It would be cleaner to move all commands that affect one subsystem into bb::Subsystem::handleConsoleCommand() (currently purely virtual); this could take start/stop/restart/set etc. On toplevel we could keep commands affecting all subsystems.

bjoerngiesler commented 1 year ago

Move done, needs testing

bjoerngiesler commented 1 year ago

Testing done. And it's a LOT cleaner and much less code <3