Re: [gdal-dev] GetFeatureCount() bForce set to FALSE

2014-09-08 Thread Martin Landa
Hi, 2014-09-09 4:54 GMT+02:00 David Strip : > 83 int GetFeatureCount(int = TRUE); > > > ie, you have not changed the default to FALSE as you intended. If this is the > code you're compiling, it would certainly explain your problem. no, of course I changed the code, ie. int

Re: [gdal-dev] GetFeatureCount() bForce set to FALSE

2014-09-08 Thread David Strip
On 9/8/2014 8:15 AM, Martin Landa wrote: Hi all, I was trying to change a default value of bForce in OGRVFKLayer::GetFeatureCount() to FALSE [1], but when I debug eg. ogrinfo, it still reports bForce as TRUE (1) Breakpoint 1, OGRVFKLayer::GetFeatureCount (this=0x71a7d0, bFor

Re: [gdal-dev] GetFeatureCount() bForce set to FALSE

2014-09-08 Thread Martin Landa
Hi Even, 2014-09-08 16:22 GMT+02:00 Even Rouault : > My suggestion would be that you remove the (|| bForce) test in your > implementation, if the poDataBlock->GetFeatureCount() is reliable. that make sense to me, thanks. Martin -- Martin Landa * http://geo.fsv.cvut.cz/gwiki/Landa __

Re: [gdal-dev] GetFeatureCount() bForce set to FALSE

2014-09-08 Thread Even Rouault
Martin, I'm far from being a C++ expert, but I don't think it makes sense to change the default value of an optional parameter of a virtual method in a child class of the base class. http://svn.osgeo.org/gdal/trunk/gdal/ogr/ogrsf_frmts/ogrsf_frmts.h : virtual int GetFeatureCount( int bFor

[gdal-dev] GetFeatureCount() bForce set to FALSE

2014-09-08 Thread Martin Landa
Hi all, I was trying to change a default value of bForce in OGRVFKLayer::GetFeatureCount() to FALSE [1], but when I debug eg. ogrinfo, it still reports bForce as TRUE (1) Breakpoint 1, OGRVFKLayer::GetFeatureCount (this=0x71a7d0, bForce=1) at ogrvfklayer.cpp:162 Any idea what could be wrong? Th