Re: [PATCH][C++] Fix PR52841

2012-06-06 Thread Fabien Chêne
2012/6/6 Jason Merrill : [...] > Often if we continue parsing something on the assumption that the previous > bits are correct it leads to problems; that's all there is to this bug. >  When we see > >  using sat::Solvable::bool_type; > > cp_parser_alias_declaration eats "sat" and then tries to pars

Re: [PATCH][C++] Fix PR52841

2012-06-06 Thread Jason Merrill
On 06/06/2012 04:09 AM, Fabien Chêne wrote: Out of curiosity, do you happen to know what kind of problem was triggered here ? Shouldn't we fix the underlying issue as well ? Unless that the canonical way to solve it be to simply return earlier... Often if we continue parsing something on the as

Re: [PATCH][C++] Fix PR52841

2012-06-06 Thread Fabien Chêne
2012/6/5 Jason Merrill : > OK. Out of curiosity, do you happen to know what kind of problem was triggered here ? Shouldn't we fix the underlying issue as well ? Unless that the canonical way to solve it be to simply return earlier... Thanks, -- Fabien

Re: [PATCH][C++] Fix PR52841

2012-06-05 Thread Jason Merrill
OK. Jason

[PATCH][C++] Fix PR52841

2012-06-04 Thread Richard Guenther
We are using the following patch from Fabien in our tree to fix a rejects-valid bug quite appearant in our own codebase. Re-bootstrap and regtest on the branch head currently running, ok if that succeeds? Thanks, Richard. 2012-06-04 Fabien Chêne PR c++/52841 * parser.c (cp_p