Opened 11 years ago

Closed 10 years ago

#6291 closed Bugs (fixed)

Distinct parser parses incompletely

Reported by: Jeff Trull <edaskel@…> Owned by: Joel de Guzman
Milestone: To Be Determined Component: spirit
Version: Boost 1.46.0 Severity: Problem
Keywords: distinct qi repository Cc:

Description

Beginning in 1.46, the repository "distinct" parser, when preceded by another parser, truncates its synthesized string attribute to the first character. Test case attached passes on 1.45 and fails on 1.46 through 1.48.

Attachments (2)

keyword_issue.cpp​ (1.4 KB ) - added by Jeff Trull <edaskel@…> 11 years ago.
minimal test case
disctinct_patch.patch​ (2.2 KB ) - added by anonymous 10 years ago.

Download all attachments as: .zip

Change History (7)

by Jeff Trull <edaskel@…>, 11 years ago

Attachment: keyword_issue.cpp​ added

minimal test case

comment:1 by jeff.trull@…, 10 years ago

This issue is still present in Boost 1.52

comment:2 by thomas@…, 10 years ago

Here's a patch to solve this issue. It should work on any boost version. The problem was that the container handling modifications made in 1.46 weren't taken into account for the components sleeping in the repository.

I didn't look at the other components, but it could very well be that they also lack the container support.

by anonymous, 10 years ago

Attachment: disctinct_patch.patch​ added

comment:3 by Jeff Trull <edaskel@…>, 10 years ago

Thanks Thomas! It works. Can we get this in trunk?

comment:4 by Joel de Guzman, 10 years ago

Awesome! I'll apply the patch into trunk as soon as I can. Thanks, Thomas!

comment:5 by Joel de Guzman, 10 years ago

Resolution: → fixed
Status: new → closed
Note: See TracTickets for help on using tickets.