This change is large but is almost entirely minor formatting and maintainability fixes.
For example, using let instead of var. Also, pulling assignments out of conditionals.
This has two primary benefits. First, the code is easier to read and understand, though it is a bit longer. Second, my IDE (Webstorm) no longer complains about all these minor issues. This makes it easier to see regressions.
No semantics should be changed by these commits. However, it is possible because var has a wider scope than let.
This change is large but is almost entirely minor formatting and maintainability fixes.
For example, using
letinstead ofvar. Also, pulling assignments out of conditionals.This has two primary benefits. First, the code is easier to read and understand, though it is a bit longer. Second, my IDE (Webstorm) no longer complains about all these minor issues. This makes it easier to see regressions.
No semantics should be changed by these commits. However, it is possible because
varhas a wider scope thanlet.