Tom Lane wrote:
I'm surprised you did that much work and did not fix things to
distinguish ListCell * from List * ... we're gonna have to make
another pass over all these places to do that.

That would mean changing _every_ usage of foreach(), and a bunch of other places besides; plus, once one changes an iteration variable to be ListCell*, you get compiler warnings if you don't disable the list compat API for the entire file. Since the hope was to make the changes minimally invasive, I figured I could do this once the initial work is in CVS.

Also, I'm not so worried about changes that are effectively s/List/ListCell/ in a bunch of places -- they can largely be automated...

Offer of help is still open, if you want to try to divvy up the
remaining work.

I'd be happy to accept some help (Alvaro was kind enough to offer as well), if we can figure out a reasonable way to divide the work up (say, if this work were being done in a CVS branch... too late for that now though). Any thoughts on how best to do this?

At any rate, once the initial work is in CVS then anyone can submit patches to update particular parts of the tree to use the new API.

I really want to see this get in.

As do I.

BTW, should I take your lack of comment on linitial() as "no objection"?


---------------------------(end of broadcast)--------------------------- TIP 6: Have you searched our list archives?

Reply via email to