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.

Sign up for GitHub

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

Jump to bottom

Static variable "pending" in DebounceEvent gets mixed between instances #17

Closed
xoseperez opened this issue Dec 25, 2016 · 1 comment
Closed

Static variable "pending" in DebounceEvent gets mixed between instances #17

xoseperez opened this issue Dec 25, 2016 · 1 comment
Labels
bug major

Comments

@xoseperez
Copy link
Owner

As reported by Gianluca Barbaro:

There's a logic flaw in DebounceEvent::loop(): we declare a static bool pending variable, but it gets mixed up if we have more than 1 button.
@xoseperez
Copy link
Owner Author

Fix #17. Moved static variable 'pending' to class variable '_clicked'

xoseperez added a commit that referenced this issue Jan 24, 2018
@xoseperez
Fix #17. Moved static variable 'pending' to class variable '_clicked'
3df5cef
xoseperez pushed a commit that referenced this issue Jan 24, 2018
@twomugs @xoseperez
Merged in chris-ward/espurna-cw/dev (pull request #17)
bd1c6f0 
Added optional gamma correction for light

Approved-by: Xose Pérez <xose.perez@gmail.com>
@erkgh erkgh mentioned this issue Feb 5, 2018
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug major
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@xoseperez