I said: > Anyway, get_last_socket_error() seems exactly equivalent to our macro > SOCK_ERRNO in libpq/libpq-int.h. AFAICT, the uses of it in our > code are these:
> /home/postgres/pgsql/src/backend/libpq/be-secure.c: > errno = get_last_socket_error(); > /home/postgres/pgsql/src/backend/libpq/be-secure.c: > errno = get_last_socket_error(); > /home/postgres/pgsql/src/interfaces/libpq/fe-secure.c: > SOCK_ERRNO = get_last_socket_error(); > /home/postgres/pgsql/src/interfaces/libpq/fe-secure.c: > SOCK_ERRNO = get_last_socket_error(); > and I think every one of these is bogus and should be removed. > At best they're no-ops. > If that is the only use of e_os.h stuff then we can stop including > the file ... These were indeed the only uses of e_os.h symbols. I have removed them and the #includes. regards, tom lane ---------------------------(end of broadcast)--------------------------- TIP 1: subscribe and unsubscribe commands go to [EMAIL PROTECTED]