Proto for Boost mailing list page: Re: [proto] Adding stuff in proto operator

Subject: Re: [proto] Adding stuff in proto operator
From: Eric Niebler (eric_at_[hidden])
Date: 2010-12-29 10:04:24

On 12/29/2010 5:40 AM, Joel Falcou wrote:
> Error found.
> 
> The problem was in the and_impl transform. It uses comma operator to
> chain calls to each and_ alternatives.
> However, when this is used in a grammar used as a Generator, it enters a
> subtle infinite loop as each comma
> want to build an expression with the newly generated expression.
<snip>

The problem is that in this expression, Proto's overloaded comma
operator is considered. It shouldn't be. I was being a little too cute
when I used the comma operator like this.

> Is this fix acceptable or am I doing something wrong all together ?
> If yes, Eric, any objections that I merge this into trunk ?

I made a few adjustments and committed it myself. Thanks, Joel!

-- 
Eric Niebler
BoostPro Computing
http://www.boostpro.com