Skip to content
New issue

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

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

Already on GitHub? # to your account

adds tickCap, cleans up timer module, adds docs #97

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

mgh
Copy link
Contributor

@mgh mgh commented Jan 28, 2016

tickCap allows applications to specify a cap for tick values
separately from the maxTick property.

cleanup:

  • timer.maxTick can now be set to control the value of maxTick
  • timer.debug was broken and unused, now it is removed
  • global variable ok was removed
  • jsdoc strings added

Martin Hunt added 2 commits January 28, 2016 17:39
`tickCap` allows applications to specify a cap for tick values
separately from the maxTick property.

cleanup:
 - `timer.maxTick` can now be set to control the value of `maxTick`
 - `timer.debug` was broken and unused, now it is removed
 - global variable `ok` was removed
 - jsdoc strings added
* - `100 ⟶ onTick(100)`
* - `120 ⟶ onTick(100)`
* - `9999 ⟶ onTick(100)`
* - `10000 ⟶ onTick(10000)`
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Was this supposed to be onTick(100) as well?

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants