PGEO driver does not support Multipolygons

[aboudreault]

The pgeo driver does not support Multipolygons during the reading. It always returns a simple polygon to OGR.

Solution: Use OGRGeometryFactory::organizePolygons during the reading to smartly return the appropriate shape.

[aboudreault]

The pgeo patch

[aboudreault]

Could anyone review this patch please? Then I'll commit and backport it in branch 1.7.

[Even Rouault]

Just from review, the patch looks fine to me (judging from its proximity with similar code in shape2ogr.cpp). I see you also use the ONLY_CCW hint for organizePolygons(). So I guess there's the same convention in PGeo like in shapefile driver that holes are counterclockwise defined ?

I haven't tested, but I assume you have, so go ahead ;-)

[aboudreault]

I'm not sure for the counterclockwise. I've found this in the esri doc: Exterior rings are oriented in a clockwise direction while interior rings are oriented counterclockwise. But I think it's only for ArcGIS. I'm going to check further for the pgeo format.

[aboudreault]

Since I've been able to read a polygon that contains an inner ring using the counterclockwise method, I'd assume they followed the same convention.

[aboudreault]

Well, I thougt I had the commit right since I had already done something in ogr, apparently no. I'll let you commit the patch.

[Even Rouault]

Patch applied in trunk (r19778) and in branches/1.7 (r19779)