Re: [boost] [ANN] Phoenix-2 prerelease

From: David Abrahams (dave_at_[hidden])
Date: 2005-11-04 11:45:58

Joel de Guzman <joel_at_[hidden]> writes:

> Comments, feedback very welcome!

  template <typename Eval>
  struct actor : Eval
  {
      return_type
      operator()() const;

      template <typename T0>
      return_type
      operator()(T0& _0) const;

      template <typename T0, typename T1>
      return_type
      operator()(T0& _0, T1& _1) const;

      //...
  };

Is that really what the declaration of actor looks like?  I have a
hard time believing that there's an identifier called return_type
involved.  IMO this is just confusing.  Are you sure you want to show
any code here at all?

Oh! and you're using _0, _1, et. al. as parameter names!  Wow, really
confusing!

-- 
Dave Abrahams
Boost Consulting
www.boost-consulting.com