[gdal-dev] kml driver ndepth

[Even Rouault]

On mercredi 26 avril 2017 14:23:51 CEST Kurt Schwehr wrote:
> Even,
> 
> In the KML driver, https://trac.osgeo.org/gdal/changeset/16908 introduced
> some debugging code with nDepth.  Looks like it has never been used since.
> I converted the comment out code to be inside DEBUG_VERBOSE, but I missed
> the --nDepth.  Are you okay if we just remove that unused debugging code?
> 
> https://trac.osgeo.org/gdal/browser/trunk/gdal/ogr/ogrsf_frmts/kml/kmlnode.c
> pp#L218
> 
> ...
> 
> https://trac.osgeo.org/gdal/browser/trunk/gdal/ogr/ogrsf_frmts/kml/kmlnode.c
> pp#L325
> 
> //nDepth --;

Why not just moving it inside the #ifdef DEBUG_VERBOSE ?
> 
> -kurt


-- 
Spatialys - Geospatial professional services
http://www.spatialys.com
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.osgeo.org/pipermail/gdal-dev/attachments/20170426/452af00b/attachment.html>