Re: [HACKERS] Dllist public/private part

2003-07-02 Thread Bruce Momjian
Mendola Gaetano wrote: > "Bruce Momjian" <[EMAIL PROTECTED]> wrote: > > Mendola Gaetano wrote: > > I certainly would like to see Dllist removed too. > > > > > This mean that is waste of time work on dllist. > > > I seen that exist a TODO list about "features", > > > exist a list about: "code to op

Re: [HACKERS] Dllist public/private part

2003-07-02 Thread Mendola Gaetano
"Bruce Momjian" <[EMAIL PROTECTED]> wrote: > Mendola Gaetano wrote: > I certainly would like to see Dllist removed too. > > > This mean that is waste of time work on dllist. > > I seen that exist a TODO list about "features", > > exist a list about: "code to optimize" ? > > What TODO item where y

Re: [HACKERS] Dllist public/private part

2003-07-01 Thread Bruce Momjian
Mendola Gaetano wrote: > "Tom Lane" <[EMAIL PROTECTED]> wrote: > > "Mendola Gaetano" <[EMAIL PROTECTED]> writes: > > > I'm improving the Dllist in these direction: > > > > AFAIR, catcache.c is the *only* remaining backend customer for Dllist, > > and so any improvement for Dllist that breaks catcac

Re: [HACKERS] Dllist public/private part

2003-07-01 Thread Mendola Gaetano
"Tom Lane" <[EMAIL PROTECTED]> wrote: > "Mendola Gaetano" <[EMAIL PROTECTED]> writes: > > I'm improving the Dllist in these direction: > > AFAIR, catcache.c is the *only* remaining backend customer for Dllist, > and so any improvement for Dllist that breaks catcache is hardly an > improvement, no?

Re: [HACKERS] Dllist public/private part

2003-06-30 Thread Tom Lane
"Mendola Gaetano" <[EMAIL PROTECTED]> writes: > I'm improving the Dllist in these direction: AFAIR, catcache.c is the *only* remaining backend customer for Dllist, and so any improvement for Dllist that breaks catcache is hardly an improvement, no? > 1) Avoid "if" statements in insertion/remove p

[HACKERS] Dllist public/private part

2003-06-30 Thread Mendola Gaetano
I'm improving the Dllist in these direction: 1) Avoid "if" statements in insertion/remove phase, for instance now the AddHeader appear like this: void DLAddHead(Dllist *l, Dlelem *e) { Dlelem *where = l->dll_master_node->dle_next; e->dle_next = where; e->dle_prev = where->dle_prev; w