We looking at the code and saw updatePins(void) and updatePins(uint8_t data, uint8_t clock). It looks like they are both assuming you are switching from SW to HW and HW to SW respectively, I imagine they should both check like in the destructor versus assuming that since the user could be switching from SW pins to other SW pins, etc.
caternuson
commented
2 years ago
We looking at the code and saw updatePins(void) and updatePins(uint8_t data, uint8_t clock). It looks like they are both assuming you are switching from SW to HW and HW to SW respectively, I imagine they should both check like in the destructor versus assuming that since the user could be switching from SW pins to other SW pins, etc.