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 >> d...@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 > d...@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. _______________________________________________ Dev mailing list d...@lists.osgeo.org http://lists.osgeo.org/mailman/listinfo/openlayers-dev