Modify

Ticket #6585 (closed Patches: fixed)

Opened 2 years ago

Last modified 2 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:

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

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

Change History

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

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

Patch for WKT parsing

comment:1 Changed 2 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.

View

Add a comment

Modify Ticket

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


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

 
Note: See TracTickets for help on using tickets.