Re: [Bug Fix] ECPG: could not use set xxx to default statement - Mailing list pgsql-hackers

From Tom Lane
Subject Re: [Bug Fix] ECPG: could not use set xxx to default statement
Date
Msg-id 29824.1550552701@sss.pgh.pa.us
Whole thread Raw
In response to Re: [Bug Fix] ECPG: could not use set xxx to default statement  (Tom Lane <tgl@sss.pgh.pa.us>)
Responses Re: [Bug Fix] ECPG: could not use set xxx to default statement  (Michael Meskes <meskes@postgresql.org>)
RE: [Bug Fix] ECPG: could not use set xxx to default statement  ("Higuchi, Daisuke" <higuchi.daisuke@jp.fujitsu.com>)
List pgsql-hackers
I wrote:
> "Higuchi, Daisuke" <higuchi.daisuke@jp.fujitsu.com> writes:
>> [ missing semicolon in gram.y breaks ecpg parsing of same construct ]

> That's pretty nasty.  The fix in gram.y is certainly needed, but I'm
> unexcited by the regression test additions you propose.  What I really
> want to know is why a syntax error in gram.y wasn't detected by any
> of the tools we use,

Ugh ... the Bison NEWS file has this:

* Changes in version 1.875, 2003-01-01:
  ...
  - Semicolons are once again optional at the end of grammar rules.
    This reverts to the behavior of Bison 1.33 and earlier, and improves
    compatibility with Yacc.

I'd remembered how we had to run around and insert semicolons to satisfy
Bison 1.3-something, and supposed that that restriction still held.
But it doesn't.  It seems though that our conversion script for creating
preproc.y depends on there being semicolons.

I think we need to fix that script to either cope with missing semicolons,
or at least complain about them.  Too tired to look into how, right now.

            regards, tom lane


pgsql-hackers by date:

Previous
From: Michael Paquier
Date:
Subject: Re: Offline enabling/disabling of data checksums
Next
From: Tom Lane
Date:
Subject: Re: Prevent extension creation in temporary schemas