Superfluous "limit" in Layer.XYZ

[Joto]

The getURL function in Layers.XYZ contains the line

var limit = Math.pow(2, z);

But this limit is never used. It is probably a left-over from a check that the x/y-coordinates are within the limits. (OpenLayers tends to generate tile requests for tiles numbered -1 etc.) This line should either be removed or the check should be added.

[fredj]

remove unused variable

[fredj]

tests pass, please review.

[fredj]

should we change the milestone to 2.9 and commit now or wait after the 2.9 release ?

[bartvde]

Hi fredj, since this is not urgent, I would like to leave it at the 2.10 milestone, otherwise 2.9 will never get done.

It can be committed now in trunk, but I won't pull it up to the 2.9 branch.

[bartvde]

Okay since RC2 has not yet been tagged etc. and the vote is still out, I've changed my mind to also include this in RC2. So please commit in trunk with state pullup.

[bartvde]

Fredj, the commit for this one still needs to be done right?

[fredj]

it's commited: r10182 (but the pullup keyword doesn't worked)

[bartvde]

Pulled up in r10185 to the 2.9 branch