Modify

Opened 6 years ago

Closed 6 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@…> 6 years ago.
wkt.patch (2.8 KB) - added by Aleksey Tulinov <aleksey.tulinov@…> 6 years ago.
Patch for WKT parsing

Download all attachments as: .zip

Change History (3)

Changed 6 years ago by Aleksey Tulinov <aleksey.tulinov@…>

Attachment: test1.cpp added

Changed 6 years ago by Aleksey Tulinov <aleksey.tulinov@…>

Attachment: wkt.patch added

Patch for WKT parsing

comment:1 Changed 6 years ago by Barend Gehrels

Milestone: To Be DeterminedBoost 1.50.0
Resolution: fixed
Status: newclosed
Type: BugsPatches

Thanks for this patch.

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

Modify Ticket

Change Properties
Set your email in Preferences
Action
as closed The owner will remain Barend Gehrels.
The resolution will be deleted.

Add Comment


E-mail address and name can be saved in the Preferences.

 
Note: See TracTickets for help on using tickets.