search

flybeyond / atrias

Automatically exported from code.google.com/p/atrias
0 stars 0 forks source link

Split medulla identification for each type into functions. #117

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago 
Currently, the medullasInit() function in the EtherCAT connector's 
MedullaManager class has lots of indentation. Further, we are identifying 
product codes w/ if/else blocks so as to not nest switch statements.

Each medulla type should have its own identification/initialization function 
with a switch statement. This would probably be a bit cleaner (and eliminate a 
lot of indentation).

This is just a minor code cleanup.

Original issue reported on code.google.com by jrvanwhy on 7 Sep 2012 at 9:25

GoogleCodeExporter commented 9 years ago 
This can wait until after we've checked out the Orocos SOEM-based master, as if 
we switch to that then the MedullaManager will go away.

Original comment by jrvanwhy on 7 Sep 2012 at 9:26

GoogleCodeExporter commented 9 years ago 
Done.

Original comment by jrvanwhy on 22 Oct 2012 at 3:12