The Metro class should not be a subclass of DigitalSource cause it does not make sense to have rose() fell() ... functions

SofaPirate / Plaquette

Object-oriented Arduino library for creative physical computing

https://sofapirate.github.io/Plaquette/

GNU General Public License v3.0

15 stars 4 forks source link

The Metro class should not be a subclass of DigitalSource cause it does not make sense to have rose() fell() ... functions #97

Open sofian opened 7 months ago

sofian commented 7 months ago

We should evaluate the advantage of it being a subclass. If it is useful, then at least make the rose() fell() etc. functions private. Could be related to #88 eg. if we choose to put the state change functions as some kind of separate class/interface like "StateChangeable" which could be appended to many classes.