If you want to keep the code, that's what I was going to do, but I'd rather just remove the code since that code looks like it was useful just during initial development.
On Wed, Apr 26, 2017 at 2:31 PM, Even Rouault <[email protected]> wrote: > 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 > -- -- http://schwehr.org
_______________________________________________ gdal-dev mailing list [email protected] https://lists.osgeo.org/mailman/listinfo/gdal-dev
