Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Event sensor: do not detach existing interrupts #1649

Merged
merged 2 commits into from
Mar 25, 2019

Conversation

mcspr
Copy link
Collaborator

@mcspr mcspr commented Mar 23, 2019

fix #1557

  • allow multiple interrupt handlers. check if other EventSensor is already handling and detach.
    technically, it is not required to detach before reattachment, but while changing this i though it may be beneficial to still keep it but wire into the other sensors to track whats where. attachInterrupt from something else will detach EventSensor handler silently
  • slightly faster method for debounce - cpu clock count instead of millis

@xoseperez xoseperez merged commit 7c93066 into xoseperez:dev Mar 25, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Multiple Event Sensors not working
2 participants