Re: Improve list manipulation in several places - Mailing list pgsql-hackers

From Ranier Vilela
Subject Re: Improve list manipulation in several places
Date
Msg-id CAEudQArQSosP9qNR62M6_44t_iicQ4f9JAWmB5nriRccx-TfeQ@mail.gmail.com
Whole thread Raw
In response to Improve list manipulation in several places  (Richard Guo <guofenglinux@gmail.com>)
Responses Re: Improve list manipulation in several places
Re: Improve list manipulation in several places
List pgsql-hackers
Em sex., 21 de abr. de 2023 às 04:34, Richard Guo <guofenglinux@gmail.com> escreveu:
There was discussion in [1] about improvements to list manipulation in
several places.  But since the discussion is not related to the topic in
that thread, fork a new thread here and attach a patch to show my
thoughts.

Some are just cosmetic changes by using macros.  The others should have
performance gain from the avoidance of moving list entries.  But I doubt
the performance gain can be noticed or measured, as currently there are
only a few places affected by the change.  I still think the changes are
worthwhile though, because it is very likely that future usage of the
same scenario can benefit from these changes.
+1

Perhaps list_delete_nth_cell needs to check NIL too?
+ if (list == NIL)
+ return NIL;

+lcons_copy(void *datum, const List *list)
+lappend_copy(const List *list, void *datum)
list param pointer can be const here not?

regards,
Ranier Vilela

pgsql-hackers by date:

Previous
From: Daniel Gustafsson
Date:
Subject: Re: duplicate function declaration in multirangetypes_selfuncs.c
Next
From: Pavel Borisov
Date:
Subject: Re: duplicate function declaration in multirangetypes_selfuncs.c