Skip to content
This repository was archived by the owner on Apr 12, 2021. It is now read-only.

Small style fixes + Remove dead code#32

Open
antja0 wants to merge 2 commits intoketanhwr:masterfrom
antja0:master
Open

Small style fixes + Remove dead code#32
antja0 wants to merge 2 commits intoketanhwr:masterfrom
antja0:master

Conversation

@antja0
Copy link

@antja0 antja0 commented Oct 6, 2019

No description provided.

@antja0
Copy link
Author

antja0 commented Oct 6, 2019

Hi!
I just wanted to help a bit in the spirit of HacktoberFest!
I hope it's ok 😃

I also leave some codebase improvement suggestions here in case you want to create a ticket for someone else / do it yourself:

  1. Replace most of var with let and const for some memory optimization (on the downside might not work on very old browsers)
  2. Place 'use strict'; on top of the file. https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Strict_mode

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant