Le dimanche 06 octobre 2013 22:21:52, Tamas Szekeres a écrit :
> Hi Jürgen,
> 
> The commit you're referring to, doesn't seem contain the handling of the
> bExactMatch parameter at the driver. At the moment I consider this a bit
> redundant if calling OGRLayer::FindFieldIndex with bExactMatch = TRUE does
> the same as OGRLayer::GetFieldIndex. In this regard I'd be in favour of
> using the same signature for both functions. In ogr2ogr we could then do
> something like:
> 
> int iDstField = bExactFieldNameMatch?
> poDstLayer->GetFieldIndex(poSrcFieldDefn->GetNameRef()) :
> poDstLayer->FindFieldIndex(poSrcFieldDefn->GetNameRef());

Tamas,

Actually, it was committed in https://github.com/jef-
n/gdal/commit/a12311e56cf09d9deb15a18e93704377d1627c12 following a suggestion 
from mine in the discussion of the pull request.

My rationale was that we have already a few methods that take an approx flag 
(e.g. GetFeatureCount(), CreateField() ), but I don't feel very strongly for 
one way or the other one.

Even

-- 
Geospatial professional services
http://even.rouault.free.fr/services.html
_______________________________________________
gdal-dev mailing list
gdal-dev@lists.osgeo.org
http://lists.osgeo.org/mailman/listinfo/gdal-dev

Reply via email to