On May 9, 2007, at 5:32 AM, Jim Jagielski wrote:
On May 9, 2007, at 8:18 AM, Roy T. Fielding wrote:
On May 8, 2007, at 11:25 AM, [EMAIL PROTECTED] wrote:
+#define USE_ALTERNATE_IS_CONNECTED 1
+
+#if !defined(APR_MSG_PEEK) && defined(MSG_PEEK)
+#define APR_MSG_PEEK MSG_PEEK
+#endif
+
+#if USE_ALTERNATE_IS_CONNECTED && defined(APR_MSG_PEEK)
Huh? Why are we polluting macro space with useless defines?
The USE_ALTERNATE_IS_CONNECTED is to allow that section
to be bypassed (although it should be protected
by a "ifndef USE_ALTERNATE_IS_CONNECTED" to allow
compile time changes I'm guessing).
Well, that would at least give it some reason to exist. If we know
that new function works, then remove the defines and the old function.
If it only works when APR_MSG_PEEK is known by APR, then that is
the only #if needed (and we should not need to define our own).
If we don't know if it works, then -1 on the backport.
The APR_MSG_PEEK
is so when APR is updated to reflect the
existence of APR_MSG_PEEK, we don't need to touch
this code. Finally, the older version assumed
that MSG_PEEK was defined and the compile would
barf if not; this avoid that.
Well, the code won't work anyway if APR doesn't know about MSG_PEEK.
+ status = apr_socket_recvfrom(&unused, socket, APR_MSG_PEEK,
+ &buf[0], &len);
presumably means something to APR, since otherwise we will lose data.
....Roy
