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.
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
Proto list run by eric at boostpro.com