GDAL_DEFAULT_WMS_CACHE_DEPTH

[vindoctor]

I'm hoping you will add this to the current 2.0 trunk. You can copy paste what I have below and it will keep all the old behaviors with the addition to being able to set the cache depth as an option if not in the xml file.

CPLErr GDALWMSCache::Initialize(CPLXMLNode *config) {

const char *xmlcache_path = CPLGetXMLValue(config, "Path", NULL); const char *usercache_path = CPLGetConfigOption("GDAL_DEFAULT_WMS_CACHE_PATH", NULL); if(xmlcache_path) {

m_cache_path = xmlcache_path;

} else {

if(usercache_path) {

m_cache_path = usercache_path;

} else {

m_cache_path = "./gdalwmscache";

}

}

const char *cache_depth = CPLGetXMLValue(config, "Depth", NULL);

if(!cache_depth) {

cache_depth = CPLGetConfigOption("GDAL_DEFAULT_WMS_CACHE_DEPTH", NULL);

}

if(cache_depth) {

m_cache_depth = atoi(cache_depth);

} else {

m_cache_depth = 2;

}

const char *cache_extension = CPLGetXMLValue(config, "Extension", ""); m_postfix = cache_extension;

return CE_None;

}

[Even Rouault]

A more detailed rationale justifying the change, the change attached as a proper patch (svn diff), and including the update of the WMS documentation page would help integrating it.

[vindoctor]

Ok. I'll work on such. As for rationale, the code in c++ has hardcoded defaults which are the best choice for the original software coder, but not for others. We use JSON abilites which means we have no way to set these defaults(as we have no XML file at times)

[Even Rouault]

Still no patch ?

[Even Rouault]

Removing obsolete milestone

[Even Rouault]

This ticket has been automatically closed because Trac is no longer used for GDAL bug tracking, since the project has migrated to GitHub. If you believe this ticket is still valid, you may file it to ​https://github.com/OSGeo/gdal/issues if it is not already reported there.