sequencing with comma does not work for boost::phoenix::bind

[Gennady Proskurin <gpr@…>]

Sample program attached. Correct behaviour, I think, is to call both functions, f() and g(). In new phoenix, only last function g() is called.

Using old phoenix from boost/spirit: c++ func2.cc Phoenix version: 2000 f g

Using new phoenix: c++ func2.cc -D USE_NEW_PHOENIX ./a.out Phoenix version: 3000 g

It seems that comma operator is not overloaded for boost::phoenix::bind (or something like that).

[John Fletcher <J.P.Fletcher@…>]

The solution to this is that an extra header is needed for Phoenix V3. The actual header is #include <boost/proto/operators.hpp> This can be included by including any of the following: #include <boost/phoenix/operator/member.hpp> #include <boost/phoenix/operator.hpp> #include <boost/phoenix.hpp>

[John Fletcher <J.P.Fletcher@…>]

I have countered the need for the extra header by adding #include <boost/proto/operators.hpp> to boost/phoenix/statement/sequence.hpp so that now the headers needed for V2 and V3 are equivalent. I have also added a test bug5715 to check correct operation.

[Kohei Takahashi]

Fixed in 1.56. ​http://melpon.org/wandbox/permlink/3QQ2uLVuioB12q08