Peter has fixed this in CVS. Thanks for the report. ---------------------------------------------------------------------------
Richard Huxton wrote: > Versions: 7.3, 7.4beta (not latest) > Applies to SQL functions, but not apparently to plpgsql (because of the > different parser, I presume). > > This is really a "doctor it hurts when I..." thing - the fact that I've never > come across it before must mean it's pretty hard to trigger. > > The first version of this function compiles fine, the second doesn't. > > CREATE OR REPLACE FUNCTION zzz_test () RETURNS text AS ' > SELECT ''hello world''::text; > -- SELECT ''goodbye world'' > ' LANGUAGE 'SQL'; > > CREATE OR REPLACE FUNCTION zzz_test () RETURNS text AS ' > SELECT ''hello world'' > -- SELECT ''goodbye world'' > ::text; > ' LANGUAGE 'SQL'; > > ERROR: parser: unterminated quoted string at or near "'hello world' > -- SELECT 'goodbye world' > ::text; > " > > It took forever to find a simple reproducible example, but the rules seem to > be: > 1. You must have quoting at the end of both adjacent lines > 2. Both lines must be within a single SQL statement. > 3. You must use the -- comment, not /* ... */ > > Originally spotted with a formulation like: > AND type1 = ''A'' > AND type2 = ''B'' > ... > I commented one test and got an error message. > > -- > Richard Huxton > Archonet Ltd > > ---------------------------(end of broadcast)--------------------------- > TIP 2: you can get off all lists at once with the unregister command > (send "unregister YourEmailAddressHere" to [EMAIL PROTECTED]) > -- Bruce Momjian | http://candle.pha.pa.us [EMAIL PROTECTED] | (610) 359-1001 + If your life is a hard drive, | 13 Roberts Road + Christ can be your backup. | Newtown Square, Pennsylvania 19073 ---------------------------(end of broadcast)--------------------------- TIP 9: the planner will ignore your desire to choose an index scan if your joining column's datatypes do not match