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

Various Code Improvements for Added Stability #12

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

Various Code Improvements for Added Stability #12

wants to merge 37 commits into from

Conversation

alexsomeoddpilot
Copy link

I've made these improvements pending some bigger shifts.

Indentation should be consistent. The dominant usage is two spaces, so I've converted all tab characters to spaces.
Strict mode ensures consistent behavior with newer browsers. Its useful in development to keep issues in check. Compilers/minifiers will remove it eventually for production.

https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Functions_and_function_scope/Strict_mode
I've reformatted the comments to a simpler, consistent format for use with Docco to create a simple documentation.
Lines should ideally be less than 85 characters
It doesn't matter immensely, but creates consistency with other comparisons.
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.

2 participants