Cool. What is your newer style? I can keep to that.


On Sun, Aug 4, 2013 at 11:18 PM, Joshua Bell <notifications@github.com>wrote:

> In map.js:
>
> > @@ -894,7 +891,7 @@ var Map;
> >
> >      // Initial z - leave room for lower/higher scale tiles
> >          z = 10 + this.max_scale - this.min_scale,
> > -        x, y, dx, dy, dw, dh,
> > +        x, y, dx, dy, dw, dh, quadrant, cx, cy, qx, qy, ax, ay,
>
> FYI, I'm gradually moving away from this up-front style of declaring
> variables in JS. It's recommended by jslint and it matches how variables
> are actually scoped in JS (i.e. they are all scoped to the function, not
> any inner blocks) but it detracts from readability.
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/inexorabletash/travellermap/pull/3/files#r5572027>
> .
>