Boost C++ Libraries: Ticket #1048: Spirit returns full = false in 1.34 if there is trailing spaces in input

status, summary changed; resolution set

biochimia — Mon, 18 Jun 2007 02:26:10 GMT

status new → closed
resolution → wontfix
summary Spirit returns full = false in 1.3.4 if there is trailing spaces in input → Spirit returns full = false in 1.34 if there is trailing spaces in input

The change in behaviour was intentional. It was made on the grounds of consistency (parsers shouldn't consume any more output than what they have matched), as well as to avoid a hang when directly parsing stdin: namely, parse() would hang waiting for a non-whitespace character at the end of a match.

You may workaround this change by adding eps_p or end_p after your grammar, as in:

my_grammar g;
my_skipper skipper;
parse(str, g >> eps_p, skipper); // 1
parse(str, g >> end_p, skipper); // 2

The 2nd option requires a full match, while the first one gets you the previous behaviour without making a full match obligatory.

Here's where the change was first announced, for context:

http://thread.gmane.org/gmane.comp.parsers.spirit.general/9839

status changed; resolution deleted

cdiggins@… — Fri, 05 Oct 2007 03:14:59 GMT

status closed → reopened
resolution wontfix

This new change in behaviour breaks the C parser example in the repository. Others in the repository may be broken as well, I haven't checked.

status changed; owner set

Marshall Clow — Tue, 27 Nov 2007 15:58:05 GMT

owner set to Joel de Guzman
status reopened → new

status changed; resolution set

Joel de Guzman — Tue, 27 Nov 2007 23:20:12 GMT

status new → closed
resolution → fixed

Author (Hartmut Kaiser) notified and web-page note on this issue added. I shall close this now.