NSPR's internal header file pr/include/md/_freebsd.h includes these lines,
which go all the way back to rev 1.1:
/* freebsd has INADDR_LOOPBACK defined, but in /usr/include/rpc/types.h,
and I didn't want to be including that.. */
#define INADDR_LOOPBACK (u_long)0x7F000001
But NSPR's public header file prinet.h contains these lines:
#if defined(FREEBSD) || defined(BSDI) || defined(QNX)
#include <rpc/types.h> /* the only place that defines INADDR_LOOPBACK */
This inconsistency is odd. NSPR makes its users include a file that it
does not always include itself.
Sun's NSPR/NSS team has received a request to change prinet.h to use the
same #define technique as used in _freebsd.h, rather than continuing to
#include <rpc/types.h> to resolve some issues with the file rpc/types.h
in recent versions of FreeBSD.
Here, I want to ask the NSPR developers: Is this a reasonable request?
Wan-Teh, what's your opinion?
This is fine.
Is this still actual?