-
-
Notifications
You must be signed in to change notification settings - Fork 464
Adds new timer functions #3683
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
Adds new timer functions #3683
Conversation
- pauseTimer - isTimerPaused
setTimerPaused is the better name I think. |
I thought so too, but i thought other people would think that sounded too weird. I'll change it later when im doing all of the other changes |
If you wish to use |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good. And since it overcome several reviews ready to merge.
Just practicing and this is something I thought of, surprised it's not already implemented.
bool pauseTimer(timer theTimer, bool paused)
bool setTimerPaused(timer theTimer, bool paused)
-- Setting paused to true pauses the timer, while false resumes it.bool isTimerPaused(timer theTimer)
-- returns true if paused