Thread: Bug in translation of DEALLOCATE PREPARE
Hi, I believe in recent interface changes changes in ecpg the translation of 'DEALLOCATE PREPARE' were missed out. This worker fine under 7.3.5. Currently EXEC SQL DEALLOCATE PREPARE foobar; is translated to { ECPGdeallocate(__LINE__, ""foobar"");} The double quotes are obviously wrong, but when I remove them the compiler says that it now expects 3 arguments to ECPGdeallocate instead of 2. I assume this is a trivial fix to ecpg, but I don't have the time right now to try to understand what ecpg does internally. I'd really appreciate it if somebody could tell me how/where ecpg needs to be fixed. Cheers, Adriaan
> -----Original Message----- > From: pgsql-interfaces-owner@postgresql.org > [mailto:pgsql-interfaces-owner@postgresql.org]On Behalf Of Michael > Meskes > Sent: Friday, May 21, 2004 9:51 AM > To: Adriaan Joubert > Cc: pgsql-interfaces@postgresql.org > Subject: Re: [INTERFACES] Bug in translation of DEALLOCATE PREPARE > > > On Sat, Apr 03, 2004 at 08:35:45AM +0300, Adriaan Joubert wrote: > > I believe in recent interface changes changes in ecpg the > > translation of 'DEALLOCATE PREPARE' were missed out. This > worker fine > > under 7.3.5. > > You're completely correct. I just committed a patch to HEAD and 7.4. > > Thanks for the report. I'm moving some code to 7.4.2, and just got bit by this problem. I've surfed the mirrors, but can't seem to find anything newer than March 8. Could someone point me to where the patch is? TIA.
Owens, Steve wrote: > > -----Original Message----- > > From: pgsql-interfaces-owner@postgresql.org > > [mailto:pgsql-interfaces-owner@postgresql.org]On Behalf Of Michael > > Meskes > > Sent: Friday, May 21, 2004 9:51 AM > > To: Adriaan Joubert > > Cc: pgsql-interfaces@postgresql.org > > Subject: Re: [INTERFACES] Bug in translation of DEALLOCATE PREPARE > > > > > > On Sat, Apr 03, 2004 at 08:35:45AM +0300, Adriaan Joubert wrote: > > > I believe in recent interface changes changes in ecpg the > > > translation of 'DEALLOCATE PREPARE' were missed out. This > > worker fine > > > under 7.3.5. > > > > You're completely correct. I just committed a patch to HEAD and 7.4. > > > > Thanks for the report. > > I'm moving some code to 7.4.2, and just got bit by this problem. I've > surfed the mirrors, but can't seem to find anything newer than March 8. > Could someone point me to where the patch is? You will have to grab the CVS tag 7.4 stable tags and use that. We will release a 7.4.3 that included that patch in the next few weeks. -- Bruce Momjian | http://candle.pha.pa.us pgman@candle.pha.pa.us | (610) 359-1001+ If your life is a hard drive, | 13 Roberts Road + Christ can be your backup. | Newtown Square, Pennsylvania19073