Opened 11 years ago

Closed 11 years ago

#6585 closed Patches (fixed)

Multipoint syntax variant

Reported by: aleksey.tulinov@… Owned by: Barend Gehrels
Milestone: Boost 1.50.0 Component: geometry
Version: Boost Development Trunk Severity: Problem
Keywords: Cc:

Description

Multipoint might be written in to variants: "MULTIPOINT ((10 40), (40 30), (20 20), (30 10))" or "MULTIPOINT (10 40, 40 30, 20 20, 30 10)".

read_wkt() does parse first variant, but doesn't parse second. I've added multipoint_parser to switch between variants depending on the format of the first point in WKT.

I've also added check_end() call to multi_parser to handle incorrect definitions like "MULTILINESTRING ((10 10, 20 20, 10 40), (40 40, 30 30, 40 20, 30 10)), (0 0, 1 1)"

Patch and test application are attached to ticket.

Attachments (2)

test1.cpp​ (2.0 KB ) - added by Aleksey Tulinov <aleksey.tulinov@…> 11 years ago.
wkt.patch​ (2.8 KB ) - added by Aleksey Tulinov <aleksey.tulinov@…> 11 years ago.
Patch for WKT parsing

Download all attachments as: .zip

Change History (3)

by Aleksey Tulinov <aleksey.tulinov@…>, 11 years ago

Attachment: test1.cpp​ added

by Aleksey Tulinov <aleksey.tulinov@…>, 11 years ago

Attachment: wkt.patch​ added

Patch for WKT parsing

comment:1 by Barend Gehrels, 11 years ago

Milestone: To Be Determined → Boost 1.50.0
Resolution: → fixed
Status: new → closed
Type: Bugs → Patches

Thanks for this patch.

The patch is applied. The tests are incorporated in the unit tests.

Note: See TracTickets for help on using tickets.