attachment set

anonymous — Wed, 14 Jul 2010 06:09:57 GMT

attachment → action_dispatch.hpp

anonymous — Wed, 14 Jul 2010 06:10:11 GMT

Hello again,

I've added a step in action_dispatch that checks the return type of phoenix actors: If it is a boolean, the return value is &&ed with the pass argument value. Here is the function:

template <typename Eval, typename Attribute, typename Context> bool operator()(

phoenix::actor<Eval> const& f, Attribute& attr, Context& context)

{

bool pass = true; typename pass_attribute<Component, Attribute>::type attr_wrap(attr);

typedef typename remove_reference<

typename Eval::template result<

phoenix::basic_environment<

typename remove_reference<Attribute>::type, typename remove_reference<Context>::type, typename remove_reference<bool>::type> >::type >::type

return_type;

return_type* ret = 0;

return call(f, attr_wrap, context, pass, ret);

}

template<typename Eval, typename Attribute, typename Context, typename Return> inline bool call(

phoenix::actor<Eval> const& f, Attribute& attr, Context& context, bool& pass, Return)

const {

f(attr, context, pass); return pass;

}

template<typename Eval, typename Attribute, typename Context> inline bool call(

phoenix::actor<Eval> const& f, Attribute& attr, Context& context, bool& pass, bool*)

const {

bool ret = f(attr, context, pass); return pass = pass && ret;

}

Joel de Guzman — Wed, 14 Jul 2010 09:25:10 GMT

This is not necessary. If you want an actor to be able to fail a parse, just give it the pass argument by using the placeholder _pass. See http://tinyurl.com/3xrfcsd. Example: p[_pass = false] will always fail parsing.

status changed; resolution set

Hartmut Kaiser — Sun, 17 Oct 2010 21:33:21 GMT

status new → closed
resolution → wontfix

Boost C++ Libraries: Ticket #4425: qi::action ignores phoenix actor return value

attachment set

status changed; resolution set