Is this patch going to be applied? ---------------------------------------------------------------------------
Andrew Dunstan wrote: > > > Dmitry Karasik wrote: > > [patch snipped] > > I have cleaned this patch somewhat by removing some bitrot that occurred > since it was submitted, and adjusting formatting to something more > closely resembling postgresql style (please remember to follow our style > in future). > > The attached works on HEAD and passes the supplied regression tests. > > But why do we have to call spi_freeplan? pltcl, which has prepared > queries, doesn't require this AFAICS. If memory leaks are an issue, > maybe we should bless the object into a class with a DESTROY method that > calls spi_freeplan automatically (not sure to do that in XS but I assume > it's possible). > > cheers > > andrew > > > > ---------------------------(end of broadcast)--------------------------- > TIP 4: Have you searched our list archives? > > http://archives.postgresql.org -- Bruce Momjian http://candle.pha.pa.us SRA OSS, Inc. http://www.sraoss.com + If your life is a hard drive, Christ can be your backup. + ---------------------------(end of broadcast)--------------------------- TIP 5: don't forget to increase your free space map settings