Changes in OGRS57Layer::GetExtent, GDAL 1.11.2

[julius6]

Hi,

can anyone explain why OGRS57Layer::GetExtent was modified to fail if the layer geometry is not set (wkbNone)? To the best of my knowledge, s57 supports mixed geometries, so having the geometry unset could be a very common case. The effect is not being able to fetch the DSID extent.

Thanks julius6

[Even Rouault]

wkbNone means no geometry at all in any feature of the layer. Mixed geometry is wkbUnknown. Normally it does not make sense to ask for the spatial extent of a layer without any geometry, and the test_ogrsf utility that checks the conformance of drivers with respect to the expected behaviour checks that. In the S57 case, GetExtent() returns the same result on all (spatial) layers, so querying any other layer will work.

[julius6]

OK. The S57 dataset is probably malformed, with no geometry assigned to DSID. Thanks julius6

[Even Rouault]

I think it is expected that the DSID layer has no geometry, since it contains a global descriptive record.

[Even Rouault]

I don't think action is needed on that. Closing