pg.ddx.io  pgsql-committers@postgresql.org mailing list archive  
help / color / mirror / Atom feed
From: Peter Eisentraut <peter@eisentraut.org>
To: pgsql-committers@lists.postgresql.org
Subject: pgsql: Remove check for accept() argument types
Date: Tue, 09 Nov 2021 15:02:29 +0000
Message-ID: <E1mkSdp-0008FH-7Y@gemulon.postgresql.org> (raw)

Remove check for accept() argument types

This check was used to accommodate a staggering variety in particular
in the type of the third argument of accept().  This is no longer of
concern on currently supported systems.  We can just use socklen_t in
the code and put in a simple check that substitutes int for socklen_t
if it's missing, to cover the few stragglers.

Reviewed-by: Andres Freund <andres@anarazel.de>
Discussion: https://www.postgresql.org/message-id/3538f4c4-1886-64f2-dcff-aaad8267fb82@enterprisedb.com

Branch
------
master

Details
-------
https://git.postgresql.org/pg/commitdiff/ee3a1a5b636b69dde33d68c428dd56b3389a4538

Modified Files
--------------
aclocal.m4                        |  1 -
config/ac_func_accept_argtypes.m4 | 78 -------------------------------------
configure                         | 82 ++++++---------------------------------
configure.ac                      |  2 +-
src/backend/libpq/auth.c          |  2 +-
src/backend/libpq/pqcomm.c        |  8 ++--
src/backend/postmaster/pgstat.c   |  4 +-
src/include/libpq/pqcomm.h        |  2 +-
src/include/pg_config.h.in        | 15 ++-----
src/include/port.h                |  4 ++
src/interfaces/libpq/fe-connect.c |  2 +-
src/port/getpeereid.c             |  4 +-
src/tools/msvc/Solution.pm        |  5 +--
13 files changed, 31 insertions(+), 178 deletions(-)



view thread (5+ messages)  latest in thread

Message-ID: <E1mkSdp-0008FH-7Y@gemulon.postgresql.org>
Permalink:  ../E1mkSdp-0008FH-7Y@gemulon.postgresql.org/
Also on:    postgresql.org/message-id/E1mkSdp-0008FH-7Y@gemulon.postgresql.org

 · 

reply

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Reply to all the recipients using the --to and --cc options:
  reply via email

  To: pgsql-committers@postgresql.org
  Cc: peter@eisentraut.org, pgsql-committers@lists.postgresql.org
  Subject: Re: pgsql: Remove check for accept() argument types
  In-Reply-To: <E1mkSdp-0008FH-7Y@gemulon.postgresql.org>

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

This inbox is served by DDX for PostgreSQL; see mirroring instructions
for how to clone and mirror all data and code used for this inbox