[OpenLayers-Dev] Re: More undefined variables on OpenLayers!

Xavier Mamano (jorix) xavier.mamano at gmail.com
Tue Mar 8 03:58:27 EST 2011


Hi Tim Schaub,

I think it would be interesting if any developer could easily use the
Closure Compiler in this way as part of the tests. 
For this reason I investigated the use of Closure and wrote #2972 and later
http://trac.osgeo.org/openlayers/ticket/3008.
(the current implementation of Closure in OL not allow this use, and should
be reviewed: does not work on Windows see #3158)

Now #3008 is Milestone Future. I think it would be interesting to implement
some of its parts in 2.11

Regards,

Xavier Mamano


Tim Schaub wrote:
> 
> On 3/7/11 2:29 PM, Xavier Mamano (jorix) wrote:
>> Hi,
>>
>> Using Closure Compiler is possible to have a report of all undefined
>> variables.
>>
>> Now this is the result:
>>
>> ...
>> ..\lib\OpenLayers\Layer\Grid.js:704: ERROR - variable tlViewPort is
>> undefined
>> tlViewPort = tlLayer.add(offsetX, offsetY);
>> ^
> 
> http://trac.osgeo.org/openlayers/changeset/11663/
> 
>> ..\lib\OpenLayers\Map.js:1692: ERROR - variable yRestriction is undefined
>> var xRestriction = yRestriction = 0;
>> . . . . . . . . . .^
> 
> http://trac.osgeo.org/openlayers/changeset/11664/
> 
>> ..\lib\OpenLayers\Renderer\SVG2.js:655: ERROR - variable component is
>> undefined
>> component = components[i];
>> ^
> 
> http://trac.osgeo.org/openlayers/changeset/11665/
> 
>> 3 error(s), 26 warning(s)
> 
> Thanks for the vigilance.
> Tim
> 
>>
>> (now is not possible get this report using "build.py -c closure")
>>
>> Xavier Mamano
>>
>>
>>
>>
>> --
>> View this message in context:
>> http://osgeo-org.1803224.n2.nabble.com/More-undefined-variables-on-OpenLayers-tp6099332p6099332.html
>> Sent from the OpenLayers Dev mailing list archive at Nabble.com.
>> _______________________________________________
>> Dev mailing list
>> Dev at lists.osgeo.org
>> http://lists.osgeo.org/mailman/listinfo/openlayers-dev
> 
> 
> -- 
> Tim Schaub
> OpenGeo - http://opengeo.org
> Expert service straight from the developers.
> _______________________________________________
> Dev mailing list
> Dev at lists.osgeo.org
> http://lists.osgeo.org/mailman/listinfo/openlayers-dev
> 


--
View this message in context: http://osgeo-org.1803224.n2.nabble.com/More-undefined-variables-on-OpenLayers-tp6099332p6115872.html
Sent from the OpenLayers Dev mailing list archive at Nabble.com.


More information about the Dev mailing list