More explict key variable names

This commit is contained in:
Tom Payne
2012-08-05 00:33:30 +02:00
parent a3b73d5fd4
commit d7623356fc
2 changed files with 18 additions and 18 deletions

View File

@@ -303,10 +303,10 @@ ol.webgl.Map.prototype.getGL = function() {
*/ */
ol.webgl.Map.prototype.getProgram = function( ol.webgl.Map.prototype.getProgram = function(
fragmentShaderObject, vertexShaderObject) { fragmentShaderObject, vertexShaderObject) {
var key = var programKey =
goog.getUid(fragmentShaderObject) + '/' + goog.getUid(vertexShaderObject); goog.getUid(fragmentShaderObject) + '/' + goog.getUid(vertexShaderObject);
if (key in this.programCache_) { if (programKey in this.programCache_) {
return this.programCache_[key]; return this.programCache_[programKey];
} else { } else {
var gl = this.getGL(); var gl = this.getGL();
var program = gl.createProgram(); var program = gl.createProgram();
@@ -321,7 +321,7 @@ ol.webgl.Map.prototype.getProgram = function(
gl.getProgramParameter(program, goog.webgl.LINK_STATUS)); gl.getProgramParameter(program, goog.webgl.LINK_STATUS));
} }
} }
this.programCache_[key] = program; this.programCache_[programKey] = program;
return program; return program;
} }
}; };
@@ -332,9 +332,9 @@ ol.webgl.Map.prototype.getProgram = function(
* @return {WebGLShader} Shader. * @return {WebGLShader} Shader.
*/ */
ol.webgl.Map.prototype.getShader = function(shaderObject) { ol.webgl.Map.prototype.getShader = function(shaderObject) {
var key = goog.getUid(shaderObject); var shaderKey = goog.getUid(shaderObject);
if (key in this.shaderCache_) { if (shaderKey in this.shaderCache_) {
return this.shaderCache_[key]; return this.shaderCache_[shaderKey];
} else { } else {
var gl = this.getGL(); var gl = this.getGL();
var shader = gl.createShader(shaderObject.getType()); var shader = gl.createShader(shaderObject.getType());
@@ -348,7 +348,7 @@ ol.webgl.Map.prototype.getShader = function(shaderObject) {
gl.getShaderParameter(shader, goog.webgl.COMPILE_STATUS)); gl.getShaderParameter(shader, goog.webgl.COMPILE_STATUS));
} }
} }
this.shaderCache_[key] = shader; this.shaderCache_[shaderKey] = shader;
return shader; return shader;
} }
}; };
@@ -579,9 +579,9 @@ ol.webgl.Map.prototype.removeLayer = function(layer) {
ol.webgl.Map.prototype.removeLayerRenderer = function(layer) { ol.webgl.Map.prototype.removeLayerRenderer = function(layer) {
var layerRenderer = goog.base(this, 'removeLayerRenderer', layer); var layerRenderer = goog.base(this, 'removeLayerRenderer', layer);
if (!goog.isNull(layerRenderer)) { if (!goog.isNull(layerRenderer)) {
var key = goog.getUid(layer); var layerKey = goog.getUid(layer);
goog.events.unlistenByKey(this.layerRendererChangeListenKeys_[key]); goog.events.unlistenByKey(this.layerRendererChangeListenKeys_[layerKey]);
delete this.layerRendererChangeListenKeys_[key]; delete this.layerRendererChangeListenKeys_[layerKey];
} }
return layerRenderer; return layerRenderer;
}; };
@@ -592,8 +592,8 @@ ol.webgl.Map.prototype.removeLayerRenderer = function(layer) {
*/ */
ol.webgl.Map.prototype.setLayerRenderer = function(layer, layerRenderer) { ol.webgl.Map.prototype.setLayerRenderer = function(layer, layerRenderer) {
goog.base(this, 'setLayerRenderer', layer, layerRenderer); goog.base(this, 'setLayerRenderer', layer, layerRenderer);
var key = goog.getUid(layer); var layerKey = goog.getUid(layer);
this.layerRendererChangeListenKeys_[key] = goog.events.listen(layerRenderer, this.layerRendererChangeListenKeys_[layerKey] = goog.events.listen(
goog.events.EventType.CHANGE, this.handleLayerRendererChange, false, layerRenderer, goog.events.EventType.CHANGE,
this); this.handleLayerRendererChange, false, this);
}; };

View File

@@ -358,11 +358,11 @@ ol.webgl.TileLayerRenderer.prototype.render = function() {
gl.bindTexture(goog.webgl.TEXTURE_2D, map.getTileTexture(tile)); gl.bindTexture(goog.webgl.TEXTURE_2D, map.getTileTexture(tile));
gl.drawArrays(goog.webgl.TRIANGLE_STRIP, 0, 4); gl.drawArrays(goog.webgl.TRIANGLE_STRIP, 0, 4);
} else { } else {
var key = goog.getUid(tile); var tileKey = goog.getUid(tile);
if (!(key in this.tileChangeListenerKeys_)) { if (!(tileKey in this.tileChangeListenerKeys_)) {
tile.load(); tile.load();
// FIXME will need to handle aborts as well // FIXME will need to handle aborts as well
this.tileChangeListenerKeys_[key] = goog.events.listen(tile, this.tileChangeListenerKeys_[tileKey] = goog.events.listen(tile,
goog.events.EventType.CHANGE, this.handleTileChange, false, this); goog.events.EventType.CHANGE, this.handleTileChange, false, this);
} }
} }