[gdal-dev] kml driver ndepth

[Kurt Schwehr]

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 <even.rouault at spatialys.com>
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
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.osgeo.org/pipermail/gdal-dev/attachments/20170426/6449e858/attachment.html>