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

Updated timer to fix crashes. #1

Merged
merged 1 commit into from Nov 20, 2017
Merged

Updated timer to fix crashes. #1

merged 1 commit into from Nov 20, 2017

Conversation

@mseneshen
Copy link
Contributor

@mseneshen mseneshen commented Dec 26, 2016

The buttons array previously had four elements, from element 0 to 3. The crashes were caused by accessing and modifying buttons[4] which was outside the bounds of the array. I fixed this by adding a fifth element to the array.

The buttons array previously had four elements, from element 0 to 3. The crashes were caused by accessing and modifying buttons[4] which was outside the bounds of the array. I fixed this by adding a fifth element to the array.
@GCaw
GCaw approved these changes Feb 21, 2017
Copy link
Owner

@GCaw GCaw left a comment

Thank you so much. Such a silly mistake on my side!

GCaw added a commit that referenced this pull request Feb 21, 2017
Thanks to mseneshen for fixing this bug: #1
@GCaw GCaw merged commit f19a6b2 into GCaw:master Nov 20, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants
You can’t perform that action at this time.