[DONE] Re: [gdal-dev] Proposed addition of OGR_L_GetName() /
OGRLayer::GetName()
Even Rouault
even.rouault at mines-paris.org
Tue Aug 17 13:56:21 EDT 2010
Implemented in trunk now.
--> Full rebuild (make clean; make) of GDAL necessary due to addition of
virtual methods.
Best regards,
Even
Le samedi 14 août 2010 20:50:18, Even Rouault a écrit :
> Le samedi 14 août 2010 19:57:52, Frank Warmerdam a écrit :
> > On Sat, Aug 14, 2010 at 11:18 AM, Even Rouault
> >
> > Even,
> >
> > I'm generally ok with this change. I would like to see it made explicit
> > that the name returned by OGRLayer::GetName() must always match the name
> > returned by the OGRFeatureDefn::GetName().
>
> Yes 100% agreed : here's what is currently stated in the doc of the method
> : "This returns the same result as GetLayerDefn()->GetName()"
>
> Is that clear enough ?
>
> > I am also wondering if there are any other aspects that should be handled
> > this way. Many applications want to show a list of laye names, and the
> > geometry type of the layer (ie. ogrinfo, QGIS, etc). Should we also be
> > making the geometry type a virtual method fetch on the layer?
>
> Yes, that makes sense although this is an information that is sometimes a
> bit harder to get than the name, however I can add easily it :
>
> OGRGeometryType OGRLayer::GetGeomType()
>
> which should return the same as GetLayerDefn()->GetGeomType()
>
> > I can see the change you propose being of benefit to essentially all the
> > RDBMS drivers.
>
> Yes, a quick look at the MYSQL driver shows that it could benefit from that
> too.
>
> > Best regards,
>
> _______________________________________________
> gdal-dev mailing list
> gdal-dev at lists.osgeo.org
> http://lists.osgeo.org/mailman/listinfo/gdal-dev
More information about the gdal-dev
mailing list