more possible dead ports cleanup

From: Peter Eisentraut <peter_e(at)gmx(dot)net>
To: pgsql-hackers <pgsql-hackers(at)postgresql(dot)org>
Subject: more possible dead ports cleanup
Date: 2012-05-06 13:04:18
Message-ID: 1336309458.6713.7.camel@vanquo.pezone.net
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

I think a few more things could removed/simplified after the recent
round of port removal:

- Remove definition of offsetof() in c.h

- (Side point, the definition of endof() in the same place isn't used
anywhere, and doesn't look safe to me, because it can go one past the
end of memory.)

- Remove strtol, strtoul, strdup from libpgport. I seem to recall these
were for nextstep at some point. Before we applied libpgport
everywhere, we were actually already going fine without these in some
parts of the source tree.

Some more risky things that can only be tested at run time;

- We could use fflush(NULL) in src/backend/postmaster/fork_process.c.

- There is some business about realloc(NULL) not working in
src/interfaces/libpq/fe-exec.c.

I suggest we do the first 2 (or 3) now, and leave the last 2 for future
cleanups.

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Tom Lane 2012-05-06 14:23:44 Re: remove dead ports?
Previous Message Hannu Krosing 2012-05-06 10:13:21 What is the current status of FOR UPDATE cursors ?