Re: pgc.l modif. has been overwritten again

Sorry for the order of the patch.

Here is the exchange I had with Micheal.

For a test case try to ecpg with test1.pgc modified this way:
....
exec sql begin declare section;
typedef char string[NAMELEN + 1];
intarray amount;
int increment=100;
char name[AMOUNT][NAMELEN+1];
char letter[AMOUNT][1];
exec sql ifdef NAMELEN;
struct name_letter_struct
{
char name[NAMELEN + 1];
int amount;
char letter;
} name_letter[AMOUNT];
exec sql endif;
struct ind_struct
{
short a;
short b;
short c;
} ind[AMOUNT];
char command[128];
....

Nicolas
----- Original Message -----
From: "Nicolas Bazin" <nbazin(at)ingenico(dot)com(dot)au>
To: "Tom Lane" <tgl(at)sss(dot)pgh(dot)pa(dot)us>
Cc: <pgsql-interfaces(at)postgresql(dot)org>
Sent: Wednesday, March 27, 2002 4:46 PM
Subject: Re: [INTERFACES] pgc.l modif. has been overwritten again

> They are not in postgresql-7.2.1.tar.gz
> I just checked the repositary with the CVS web interface and the modif are
> not in there.
> In CVS you get
> <<EOF>> {
> if ( preproc_tos > 0 )
> {
> preproc_tos = 0;
> mmerror(PARSE_ERROR, ET_FATAL, "Missing
> 'EXEC SQL ENDIF;'");
> }
> if (yy_buffer == NULL)
> yyterminate();
> else
>
>
> where is should be:
>
> <<EOF>> {
> if (yy_buffer == NULL) {
> if ( preproc_tos > 0 )
> {
> preproc_tos = 0;
> mmerror(PARSE_ERROR, ET_FATAL,
> "Missing 'EXEC SQL ENDIF;'");
> }
> yyterminate();
> } else
>
> And looking and the history of this file I don't see the modif beeing
> applied. But I know that at some stage it was applied by Bruce momjian
and
> overwritten by Mickeal Meskes ???
>
> Nicolas
> ----- Original Message -----
> From: "Tom Lane" <tgl(at)sss(dot)pgh(dot)pa(dot)us>
> To: "Nicolas Bazin" <nbazin(at)ingenico(dot)com(dot)au>
> Cc: <pgsql-interfaces(at)postgresql(dot)org>
> Sent: Wednesday, March 27, 2002 4:26 PM
> Subject: Re: [INTERFACES] pgc.l modif. has been overwritten again
>
>
> > "Nicolas Bazin" <nbazin(at)ingenico(dot)com(dot)au> writes:
> > > We need a little bit of order when several people can commit on the
> source =
> > > code, It would be nice that they update their local copy and then
> commit, o=
> > > r check the directory or check for conflicts.=20
> >
> > AFAICT these changes *are* in current sources.
> >
> > regards, tom lane
> >
> >
>