Ticket #6585 (closed Patches: fixed)

Opened 5 years ago

Last modified 5 years ago

Multipoint syntax variant

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


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.


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

Change History

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

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

Patch for WKT parsing

comment:1 Changed 5 years ago by barendgehrels

  • Status changed from new to closed
  • Type changed from Bugs to Patches
  • Resolution set to fixed
  • Milestone changed from To Be Determined to Boost 1.50.0

Thanks for this patch.

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


Add a comment

Modify Ticket

Change Properties
<Author field>
as closed
The resolution will be deleted. Next status will be 'reopened'

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

Note: See TracTickets for help on using tickets.