Boost C++ Libraries: Ticket #7165: cannot change BOOST_PHOENIX_LIMIT

m.champlon@… — Tue, 24 Jul 2012 13:01:26 GMT

It looks like a fix for this issue would be in limits.hpp to change :

#if defined(BOOST_PHOENIX_LIMIT)
#define BOOST_PROTO_MAX_ARITY BOOST_PHOENIX_LIMIT

into :

#if defined(BOOST_PHOENIX_LIMIT)
#define BOOST_PROTO_MAX_ARITY BOOST_PHOENIX_PP_ROUND_UP(BOOST_PHOENIX_LIMIT)

m.champlon@… — Mon, 30 Jul 2012 09:21:21 GMT

A bit of follow-up,

The following code fails :

#define BOOST_PHOENIX_LIMIT 20
#define BOOST_PROTO_MAX_LOGICAL_ARITY BOOST_PHOENIX_LIMIT
#include <boost/phoenix/core/argument.hpp>
void f()
{
    boost::phoenix::arg_names::arg11;
}

with :

error C2039: 'arg11' : is not a member of 'boost::phoenix::arg_names'

Adding this to the top :

#define BOOST_PHOENIX_DONT_USE_PREPROCESSED_FILES

yields :

...phoenix/core/environment.hpp(412): warning C4005: 'BOOST_PHOENIX_M0' : macro redefinition
...phoenix/support/detail/iterate_define.hpp(108) : see previous definition of 'BOOST_PHOENIX_M0'

I hope this helps, tell me if there is anything I can do to help fix all this.

Thanks !
MAT.

attachment set

Fri, 26 Apr 2013 19:47:59 GMT

attachment → boost_phoenix_7165.patch

Fri, 26 Apr 2013 19:53:25 GMT

I finally managed to write a test case and a fix for this, see boost_phoenix_7165.patch

The patch initially contained the regenerated preprocessed actor.hpp files but the server refused to upload it because it's over the size limit (it's 566 KB !). I still have it and can email it directly to you if you want.

The idea of the fix is to replace

boost::proto::and_<
  something,
  something,
  something,
  ...
>

with

boost::proto::and_< something,
boost::proto::and_< something,
boost::proto::and_< something,
  ...
> > >

Cheers, MAT.

Sun, 02 Feb 2014 16:21:58 GMT

This patch and test have now been released on develop for testing.

status changed; resolution set

Kohei Takahashi — Tue, 10 May 2016 13:53:39 GMT

status new → closed
resolution → fixed