Index: src/backend/lib/dllist.c =================================================================== RCS file: /home/alvherre/cvs/pgsql/src/backend/lib/dllist.c,v retrieving revision 1.30 diff -c -r1.30 dllist.c *** src/backend/lib/dllist.c 31 Dec 2004 21:59:48 -0000 1.30 --- src/backend/lib/dllist.c 15 Jan 2005 03:50:40 -0000 *************** *** 43,50 **** errmsg("out of memory"))); #endif } ! l->dll_head = 0; ! l->dll_tail = 0; return l; } --- 43,50 ---- errmsg("out of memory"))); #endif } ! l->dll_head = NULL; ! l->dll_tail = NULL; return l; } *************** *** 52,59 **** void DLInitList(Dllist *list) { ! list->dll_head = 0; ! list->dll_tail = 0; } /* --- 52,59 ---- void DLInitList(Dllist *list) { ! list->dll_head = NULL; ! list->dll_tail = NULL; } /* *************** *** 65,71 **** { Dlelem *curr; ! while ((curr = DLRemHead(list)) != 0) free(curr); free(list); --- 65,71 ---- { Dlelem *curr; ! while ((curr = DLRemHead(list)) != NULL) free(curr); free(list); *************** *** 88,107 **** errmsg("out of memory"))); #endif } ! e->dle_next = 0; ! e->dle_prev = 0; e->dle_val = val; ! e->dle_list = 0; return e; } void DLInitElem(Dlelem *e, void *val) { ! e->dle_next = 0; ! e->dle_prev = 0; e->dle_val = val; ! e->dle_list = 0; } void --- 88,107 ---- errmsg("out of memory"))); #endif } ! e->dle_next = NULL; ! e->dle_prev = NULL; e->dle_val = val; ! e->dle_list = NULL; return e; } void DLInitElem(Dlelem *e, void *val) { ! e->dle_next = NULL; ! e->dle_prev = NULL; e->dle_val = val; ! e->dle_list = NULL; } void *************** *** 132,140 **** l->dll_tail = e->dle_prev; } ! e->dle_next = 0; ! e->dle_prev = 0; ! e->dle_list = 0; } void --- 132,140 ---- l->dll_tail = e->dle_prev; } ! e->dle_next = NULL; ! e->dle_prev = NULL; ! e->dle_list = NULL; } void *************** *** 145,154 **** if (l->dll_head) l->dll_head->dle_prev = e; e->dle_next = l->dll_head; ! e->dle_prev = 0; l->dll_head = e; ! if (l->dll_tail == 0) /* if this is first element added */ l->dll_tail = e; } --- 145,154 ---- if (l->dll_head) l->dll_head->dle_prev = e; e->dle_next = l->dll_head; ! e->dle_prev = NULL; l->dll_head = e; ! if (l->dll_tail == NULL) /* if this is first element added */ l->dll_tail = e; } *************** *** 160,169 **** if (l->dll_tail) l->dll_tail->dle_next = e; e->dle_prev = l->dll_tail; ! e->dle_next = 0; l->dll_tail = e; ! if (l->dll_head == 0) /* if this is first element added */ l->dll_head = e; } --- 160,169 ---- if (l->dll_tail) l->dll_tail->dle_next = e; e->dle_prev = l->dll_tail; ! e->dle_next = NULL; l->dll_tail = e; ! if (l->dll_head == NULL) /* if this is first element added */ l->dll_head = e; } *************** *** 173,191 **** /* remove and return the head */ Dlelem *result = l->dll_head; ! if (result == 0) return result; if (result->dle_next) ! result->dle_next->dle_prev = 0; l->dll_head = result->dle_next; if (result == l->dll_tail) /* if the head is also the tail */ ! l->dll_tail = 0; ! result->dle_next = 0; ! result->dle_list = 0; return result; } --- 173,191 ---- /* remove and return the head */ Dlelem *result = l->dll_head; ! if (result == NULL) return result; if (result->dle_next) ! result->dle_next->dle_prev = NULL; l->dll_head = result->dle_next; if (result == l->dll_tail) /* if the head is also the tail */ ! l->dll_tail = NULL; ! result->dle_next = NULL; ! result->dle_list = NULL; return result; } *************** *** 196,214 **** /* remove and return the tail */ Dlelem *result = l->dll_tail; ! if (result == 0) return result; if (result->dle_prev) ! result->dle_prev->dle_next = 0; l->dll_tail = result->dle_prev; if (result == l->dll_head) /* if the tail is also the head */ ! l->dll_head = 0; ! result->dle_prev = 0; ! result->dle_list = 0; return result; } --- 196,214 ---- /* remove and return the tail */ Dlelem *result = l->dll_tail; ! if (result == NULL) return result; if (result->dle_prev) ! result->dle_prev->dle_next = NULL; l->dll_tail = result->dle_prev; if (result == l->dll_head) /* if the tail is also the head */ ! l->dll_head = NULL; ! result->dle_prev = NULL; ! result->dle_list = NULL; return result; } *************** *** 222,228 **** if (l->dll_head == e) return; /* Fast path if already at front */ ! Assert(e->dle_prev != 0); /* since it's not the head */ e->dle_prev->dle_next = e->dle_next; if (e->dle_next) --- 222,228 ---- if (l->dll_head == e) return; /* Fast path if already at front */ ! Assert(e->dle_prev != NULL); /* since it's not the head */ e->dle_prev->dle_next = e->dle_next; if (e->dle_next) *************** *** 236,242 **** l->dll_head->dle_prev = e; e->dle_next = l->dll_head; ! e->dle_prev = 0; l->dll_head = e; /* We need not check dll_tail, since there must have been > 1 entry */ } --- 236,242 ---- l->dll_head->dle_prev = e; e->dle_next = l->dll_head; ! e->dle_prev = NULL; l->dll_head = e; /* We need not check dll_tail, since there must have been > 1 entry */ }