menu: [#25] fix warnings and add assertions

This commit is contained in:
Vasily Davydov 2023-05-02 11:14:47 +03:00 committed by RedHawk
parent 83de15ce5b
commit bd2f6b32f1
3 changed files with 13 additions and 5 deletions

View File

@ -33,6 +33,7 @@ Event::EventPair Manager::parseEvent(Event* e)
return p; return p;
} }
} }
} }
void Manager::taskFunction() void Manager::taskFunction()

View File

@ -23,6 +23,7 @@ private:
Event::EventPair parseEvent(Event* e); Event::EventPair parseEvent(Event* e);
ThreadCommon::QueueManager* _qm; ThreadCommon::QueueManager* _qm;
Counter<EventRawData> set_point; Counter<EventRawData> set_point;
Menu* _menu;
EventRawData int_temp; EventRawData int_temp;
EventRawData ext_temp; EventRawData ext_temp;

View File

@ -6,6 +6,7 @@
*/ */
#include "Menu.h" #include "Menu.h"
#include <assert.h>
Menu::Menu() Menu::Menu()
{ {
@ -28,29 +29,34 @@ void Menu::HandleEventPair (Event::EventPair *ep)
switch(static_cast<ThreadCommon::RotaryAction>(ep->rd)/*RawData*/) switch(static_cast<ThreadCommon::RotaryAction>(ep->rd)/*RawData*/)
{ {
case ThreadCommon::RotaryAction::Right: case ThreadCommon::RotaryAction::Right:
this->HandleObj(MenuObjEvent::eRollClockWise); this->HandleObj(MenuObjEvent (MenuObjEvent::eRollClockWise));
break; break;
case ThreadCommon::RotaryAction::Left: case ThreadCommon::RotaryAction::Left:
this->HandleObj(MenuObjEvent::eRollCClockWise); this->HandleObj(MenuObjEvent (MenuObjEvent::eRollCClockWise));
break; break;
case ThreadCommon::RotaryAction::Press: case ThreadCommon::RotaryAction::Press:
this->HandleObj(MenuObjEvent::eClick); this->HandleObj(MenuObjEvent (MenuObjEvent::eClick));
break; break;
case ThreadCommon::RotaryAction::Idle: case ThreadCommon::RotaryAction::Idle:
/*I guess this is left for debugging purposes ;D*/ /*I guess this is left for debugging purposes ;D*/
this->HandleObj(MenuObjEvent (MenuObjEvent::eRefresh));
break; break;
case default: default:
assert(0);
/* Pretty damn bad code if reached here */ /* Pretty damn bad code if reached here */
break; break;
} }
break; break;
case Event::InternalTemp: case Event::InternalTemp:
// TODO // TODO
assert(0);
break; break;
case Event::ExternalTemp: case Event::ExternalTemp:
// TODO // TODO
assert(0);
break; break;
case default: default:
assert(0);
/* Manager has big issues... */ /* Manager has big issues... */
break; break;
} }