removeLayer method throws an error

Here it the code-fragment for ‘removeLayer’:

 removeLayer: function (layer) {
    var id = L.Util.stamp(layer);
    var _layer = this._getLayer(id);
    if (_layer) {
      delete this.layers[this.layers.indexOf(_layer)]; // `this.layers` should be replaced with `this._layers`
    }
    this._update();
    return this;
  }

Instead of this.layers should be this._layers i guess.

[ForgottenLords]

Calling delete on the array index is not sufficient, as it is later iterated over; delete removes the index from the array, causing the deleted index’s value to be ‘undefined’ when expecting an object. Instead, you must splice the item from the array to leave the indexes continuous.

The current pull request #50 is insufficient. I had included a fix in my pull request #46 (an unrelated feature request).

[karlitos]

Please, accept the PR to fix the issue caused by using delete on array.