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!
proto mailing list