Opened 10 years ago

Closed 10 years ago

Last modified 10 years ago

#5330 closed enhancement (fixed)

ISO WKB generation with wkbVariant option

Reported by: pramsey Owned by: evenr
Priority: normal Milestone:
Component: OGR_SF Version: unspecified
Severity: normal Keywords: wkb
Cc: Kyle Shannon

Description

As discussed on mailing list, this patch adds a wkbVariant option to exportToWkb, allowing users to optionally use the ISO WKB numbers for Z-types.

Attachments (1)

isowkb.patch​ (23.9 KB ) - added by pramsey 10 years ago.
patch passes all tests

Download all attachments as: .zip

Change History (7)

by pramsey, 10 years ago

Attachment: isowkb.patch​ added

patch passes all tests

comment:1 by pramsey, 10 years ago

Oh, patch is against SVN trunk at r26734

comment:2 by Even Rouault, 10 years ago

looks good to me. you can commit it

comment:3 by pramsey, 10 years ago

Resolution: → fixed
Status: new → closed

Committed to trunk at r26736

comment:4 by Kyle Shannon, 10 years ago

Cc: Kyle Shannon added

Paul, I didn't see anything committed in autotest/. Probably a good idea to add regression tests for the new feature.

comment:5 by Even Rouault, 10 years ago

Kyle, for now, it's internal new functionnality that has no C bindings, so it cannot be tested through Python (except maybe the importFromWkb() that uses the same function/method). This work will be triggered by the GeoPackage driver that Paul is working on.

comment:6 by Kyle Shannon, 10 years ago

Ah, understood. It will get tested in the new driver tests. Thanks.

Note: See TracTickets for help on using tickets.