Modify

Opened 5 years ago

Closed 5 years ago

#6585 closed Patches (fixed)

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 (2)

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

Download all attachments as: .zip

Change History (3)

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

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

Thanks for this patch.

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

Add Comment

Modify Ticket

Change Properties
Set your email in Preferences
Action
as closed The owner will remain barendgehrels.
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.