Re: pgsql: Remove check for accept() argument types

Поиск
Список
Период
Сортировка
От Tom Lane
Тема Re: pgsql: Remove check for accept() argument types
Дата
Msg-id 1440792.1636558888@sss.pgh.pa.us
обсуждение исходный текст
Ответ на pgsql: Remove check for accept() argument types  (Peter Eisentraut <peter@eisentraut.org>)
Ответы Re: pgsql: Remove check for accept() argument types  (Peter Eisentraut <peter.eisentraut@enterprisedb.com>)
Список pgsql-committers
Peter Eisentraut <peter@eisentraut.org> writes:
> Remove check for accept() argument types

Early returns from the buildfarm are

 gaur          | 2021-11-09 16:55:58 | auth.c:3235:17: warning: pointer targets in passing argument 6 of 'recvfrom'
differin signedness 
 gaur          | 2021-11-09 16:55:58 | pqcomm.c:722:9: warning: pointer targets in passing argument 3 of 'accept'
differin signedness 
 gaur          | 2021-11-09 16:55:58 | pqcomm.c:743:6: warning: pointer targets in passing argument 3 of 'getsockname'
differin signedness 
 gaur          | 2021-11-09 16:55:58 | pgstat.c:483:39: warning: pointer targets in passing argument 3 of 'getsockname'
differin signedness 
 gaur          | 2021-11-09 16:55:58 | pgstat.c:630:9: warning: pointer targets in passing argument 5 of 'getsockopt'
differin signedness 
 gaur          | 2021-11-09 16:55:58 | fe-connect.c:2760:11: warning: pointer targets in passing argument 5 of
'getsockopt'differ in signedness 
 gaur          | 2021-11-09 16:55:58 | fe-connect.c:2788:9: warning: pointer targets in passing argument 3 of
'getsockname'differ in signedness 

Right offhand I don't see any other animals complaining.
May I suggest that "unsigned int" would be a better choice
than "int" for socklen_t?

            regards, tom lane



В списке pgsql-committers по дате отправления:

Предыдущее
От: Robert Haas
Дата:
Сообщение: pgsql: Fix thinko in assertion in basebackup.c.
Следующее
От: Tom Lane
Дата:
Сообщение: pgsql: Doc: improve protocol spec for logical replication Type messages