Обсуждение: ECPG patches: get descriptor NULL alloc, external names
Hi Michael, these patches fix the following issues with libecpg: 1) - get descriptor didn't allocate memory if passed a NULL pointer - now get descriptor shares code with ECPGexecute regarding storage in output variables - some code cleanup, recognize timestamp without timezone 2) - make all external visible names of libecpg start with ECPG (without changing ecpg_*) 3) - rename ecpg_* to ECPG* to make the visible names look more homogenous These address the last outstanding issues I had for 7.2. Christof PS: If these patches do not apply cleanly to current sources, please tell me, I'll redo them. PPS: The first patch requires the patch (char**=0) I sent in last week (still unapplied)
Вложения
Your patch has been added to the PostgreSQL unapplied patches list at: http://candle.pha.pa.us/cgi-bin/pgpatches I will try to apply it within the next 48 hours. --------------------------------------------------------------------------- Christof Petig wrote: > Hi Michael, > > these patches fix the following issues with libecpg: > > 1) > - get descriptor didn't allocate memory if passed a NULL pointer > - now get descriptor shares code with ECPGexecute regarding storage in > output variables > - some code cleanup, recognize timestamp without timezone > > 2) > - make all external visible names of libecpg start with ECPG (without > changing ecpg_*) > > 3) > - rename ecpg_* to ECPG* to make the visible names look more homogenous > > These address the last outstanding issues I had for 7.2. > > Christof > > PS: If these patches do not apply cleanly to current sources, please > tell me, I'll redo them. > PPS: The first patch requires the patch (char**=0) I sent in last week > (still unapplied) > [ application/x-gzip is not supported, skipping... ] [ application/x-gzip is not supported, skipping... ] [ application/x-gzip is not supported, skipping... ] > > ---------------------------(end of broadcast)--------------------------- > TIP 1: subscribe and unsubscribe commands go to majordomo@postgresql.org -- Bruce Momjian | http://candle.pha.pa.us pgman@candle.pha.pa.us | (610) 853-3000 + If your life is a hard drive, | 830 Blythe Avenue + Christ can be your backup. | Drexel Hill, Pennsylvania 19026
I am told this was already added to 7.2. Thanks. --------------------------------------------------------------------------- Christof Petig wrote: > Hi Michael, > > these patches fix the following issues with libecpg: > > 1) > - get descriptor didn't allocate memory if passed a NULL pointer > - now get descriptor shares code with ECPGexecute regarding storage in > output variables > - some code cleanup, recognize timestamp without timezone > > 2) > - make all external visible names of libecpg start with ECPG (without > changing ecpg_*) > > 3) > - rename ecpg_* to ECPG* to make the visible names look more homogenous > > These address the last outstanding issues I had for 7.2. > > Christof > > PS: If these patches do not apply cleanly to current sources, please > tell me, I'll redo them. > PPS: The first patch requires the patch (char**=0) I sent in last week > (still unapplied) > [ application/x-gzip is not supported, skipping... ] [ application/x-gzip is not supported, skipping... ] [ application/x-gzip is not supported, skipping... ] > > ---------------------------(end of broadcast)--------------------------- > TIP 1: subscribe and unsubscribe commands go to majordomo@postgresql.org -- Bruce Momjian | http://candle.pha.pa.us pgman@candle.pha.pa.us | (610) 853-3000 + If your life is a hard drive, | 830 Blythe Avenue + Christ can be your backup. | Drexel Hill, Pennsylvania 19026