Re: CVS HEAD compile warning - Mailing list pgsql-hackers

From Michael Meskes
Subject Re: CVS HEAD compile warning
Date
Msg-id 20040227074105.GB3502@1
Whole thread Raw
In response to Re: CVS HEAD compile warning  (Tom Lane <tgl@sss.pgh.pa.us>)
Responses Re: CVS HEAD compile warning  (Tom Lane <tgl@sss.pgh.pa.us>)
List pgsql-hackers
On Thu, Feb 26, 2004 at 02:44:52AM -0500, Tom Lane wrote:
> I knew at the time that ecpg was the only one of our lexers in which
> echo-to-stdout could conceivably be a reasonable default rule.  But
> since flex 2.5.4 did not complain, I went ahead and committed the
> addition in ecpg as well as everyplace else.

Actually it was actively used back in earlier versions of ecpg, but I
thought it wasn't anymore.

> At this point I think our first question is whether flex 2.5.31 is
> correctly detecting a hole in the ecpg lexer rules that was missed by
> flex 2.5.4, or whether the warning is simply wrong.

Yes, that's it. The big problem seems to be that flex does not have a
way to help finding that hole.

> If the warning is valid, you need to decide whether you like the idea of
> a default spit-to-stdout action, or you'd prefer to put in an additional
> flex rule to cover the missing case.

I'd prefer to program my own default rule.

Michael
-- 
Michael Meskes
Email: Michael at Fam-Meskes dot De
ICQ: 179140304, AIM/Yahoo: michaelmeskes, Jabber: meskes@jabber.org
Go SF 49ers! Go Rhein Fire! Use Debian GNU/Linux! Use PostgreSQL!


pgsql-hackers by date:

Previous
From: Michael Meskes
Date:
Subject: Re: Thread safe connection-name mapping in ECPG. Is it required?
Next
From: "Zeugswetter Andreas SB SD"
Date:
Subject: Re: Tablespaces