search

ZachPhillipsGary / google-maps-utility-library-v3

Automatically exported from code.google.com/p/google-maps-utility-library-v3
Apache License 2.0
0 stars 0 forks source link

MarkerClusterer removeMarkers Line453 break>continue #28

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago 
The removeMarker method looks like this.

MarkerClusterer.prototype.removeMarker = function(marker, opt_nodraw) {
...
        for (var i = 0, m; m = this.markers_[i]; i++) {
            if (m == marker) {
                index = i;
                continue
            }
        }
...

this is iterating all markers instead breaking the look after the index has 
been found. Replace "continue" with "break;"

Original issue reported on code.google.com by elak....@gmail.com on 30 Jul 2010 at 12:02

GoogleCodeExporter commented 9 years ago

Original comment by lu...@google.com on 6 Oct 2010 at 12:12