Re: [HACKERS] Erroring out on parser conflicts

2008-12-03 Thread Greg Stark
On 3 Dec 2008, at 03:32 AM, Tom Lane [EMAIL PROTECTED] wrote: Bruce Momjian [EMAIL PROTECTED] writes: FYI, this is going to make it hard for developers to test CVS changes until they get their grammar cleaned up; perhaps add a comment on how to disable the check? Well, the point is

Re: [HACKERS] Erroring out on parser conflicts

2008-12-02 Thread Bruce Momjian
Peter Eisentraut wrote: On Tuesday 25 November 2008 15:09:37 Tom Lane wrote: Peter Eisentraut [EMAIL PROTECTED] writes: While hacking on parser/gram.y just now I noticed in passing that the automatically generated ecpg parser had 402 shift/reduce conflicts. (Don't panic, the parser in

Re: [HACKERS] Erroring out on parser conflicts

2008-12-02 Thread Tom Lane
Bruce Momjian [EMAIL PROTECTED] writes: FYI, this is going to make it hard for developers to test CVS changes until they get their grammar cleaned up; perhaps add a comment on how to disable the check? Well, the point is that their grammar changes are broken if that check fails, so I'm not

[HACKERS] Erroring out on parser conflicts

2008-11-25 Thread Peter Eisentraut
While hacking on parser/gram.y just now I noticed in passing that the automatically generated ecpg parser had 402 shift/reduce conflicts. (Don't panic, the parser in CVS is fine.) If you don't pay very close attention, it is easy to miss this. Considering also that we frequently have to

Re: [HACKERS] Erroring out on parser conflicts

2008-11-25 Thread Tom Lane
Peter Eisentraut [EMAIL PROTECTED] writes: While hacking on parser/gram.y just now I noticed in passing that the automatically generated ecpg parser had 402 shift/reduce conflicts. (Don't panic, the parser in CVS is fine.) If you don't pay very close attention, it is easy to miss this.

Re: [HACKERS] Erroring out on parser conflicts

2008-11-25 Thread Peter Eisentraut
On Tuesday 25 November 2008 15:09:37 Tom Lane wrote: Peter Eisentraut [EMAIL PROTECTED] writes: While hacking on parser/gram.y just now I noticed in passing that the automatically generated ecpg parser had 402 shift/reduce conflicts. (Don't panic, the parser in CVS is fine.) If you don't