ashatora/openlayers
1
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

Canvas vector / Avoid modifying the coordinates array in place

Browse Source
This commit is contained in:
Olivier Guyot
2019-09-26 15:29:38 +02:00
parent 7e4383f554
commit b89c1255f7
1 changed files with 6 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files
src/ol/renderer/Map.js
+6 -2
View File
@@ -9,7 +9,6 @@ import {inView} from '../layer/Layer.js';
import {shared as iconImageCache} from '../style/IconImageCache.js'; import {shared as iconImageCache} from '../style/IconImageCache.js';
import {compose as composeTransform, makeInverse} from '../transform.js'; import {compose as composeTransform, makeInverse} from '../transform.js';
import {renderDeclutterItems} from '../render.js'; import {renderDeclutterItems} from '../render.js';
import {add} from '../coordinate.js';
/** /**
* @abstract * @abstract
@@ -128,6 +127,8 @@ class MapRenderer extends Disposable {
return entry.value; return entry.value;
}); });
} }
const tmpCoord = [];
for (let i = 0; i < offsets.length; i++) { for (let i = 0; i < offsets.length; i++) {
for (let j = numLayers - 1; j >= 0; --j) { for (let j = numLayers - 1; j >= 0; --j) {
const layerState = layerStates[j]; const layerState = layerStates[j];
@@ -135,10 +136,13 @@ class MapRenderer extends Disposable {
if (layer.hasRenderer() && inView(layerState, viewState) && layerFilter.call(thisArg2, layer)) { if (layer.hasRenderer() && inView(layerState, viewState) && layerFilter.call(thisArg2, layer)) {
const layerRenderer = layer.getRenderer(); const layerRenderer = layer.getRenderer();
const source = layer.getSource(); const source = layer.getSource();
const coordinates = source.getWrapX() ? translatedCoordinate : coordinate;
if (layerRenderer && source) { if (layerRenderer && source) {
const callback = forEachFeatureAtCoordinate.bind(null, layerState.managed); const callback = forEachFeatureAtCoordinate.bind(null, layerState.managed);
tmpCoord[0] = coordinates[0] + offsets[i][0];
tmpCoord[1] = coordinates[1] + offsets[i][1];
result = layerRenderer.forEachFeatureAtCoordinate( result = layerRenderer.forEachFeatureAtCoordinate(
add(source.getWrapX() ? translatedCoordinate : coordinate, offsets[i]), tmpCoord,
frameState, hitTolerance, callback, declutteredFeatures); frameState, hitTolerance, callback, declutteredFeatures);
} }
if (result) { if (result) {