const doesn't technically do anything in ES5 (although as browsers are adding ES2015/ES7-8 support, that's changing), and as this is not user code, but library code, we will not be redeclaring these values at any point in the file. Since the scope is already function scope, let's make these var.

I thought I already said I'd remove const when it's about to merge

as a new PR, I'm simply filing everything we discussed "as text" where they apply to the code, so think of all of these as administrative comments (i.e.: "a change is needed here") that we can use for tracking purposes, not "I told you to change these, why haven't you changed them yet". If any of these lines still show up after the final rewrite, then we know we've forgotten something. It's a nice feature of github PRs.

var