Brad Nicholes wrote:
-1 as well. This is now causing compiler errors on NetWare. Please
revert this patch!
Can you provide an indication of exactly what broke so we
will know what to avoid in future. Or was the breakage
actually due to the the mod_cache problem reported
last night?
Thanks, Allan
mod_cache is a different issue. The compiler used to build the
netware NLMs is very sensitive to type mismatches.
@@ -3793,7 +3794,7 @@
core_net_rec *net = f-ctx;
core_ctx_t *ctx = net-in_ctx;
const char *str;
-apr_size_t len;
+apr_ssize_t len;
whoa! -1
Was this even discussed on the list? You just changed the
entire module API and introduced a dozen potential security holes.
Why on earth is it changing nvec to apr_size_t and then downcasting
its use? Why is any of this even needed?
Roy
On Oct 22, 2004, at 8:22 AM, [EMAIL
Roy T. Fielding wrote:
whoa! -1
Was this even discussed on the list? You just changed the
entire module API and introduced a dozen potential security holes.
The precursor to this patch [PATCH] WIN64: httpd API changes
was posted 10/7 so I thought we had had suitable time for
discussion. I have
The precursor to this patch [PATCH] WIN64: httpd API changes
was posted 10/7 so I thought we had had suitable time for
discussion. I have addressed the one issue that was raised.
That explains why I didn't see it -- I was in Switzerland.
There have also been several other threads on the httpd apr
Roy is right...
Willy-nilly throwing casts on data objects just to satisfy some
anal-retentive urge to not see any warnings appearing during a
compile is the absolute WRONG thing to do when it comes
to porting 32-bit code to 64-bit platforms.
The situation is NOT as simple as it was when
[EMAIL PROTECTED] writes:
jerenkrantz2002/06/26 12:45:08
Modified:.CHANGES
include ap_mmn.h httpd.h
modules/arch/win32 mod_isapi.c
modules/http http_core.c http_protocol.c http_request.c
modules/loggers
Jeff Trawick wrote:
+AP_DECLARE(void) ap_setup_make_content_type(apr_pool_t *pool);
What's up with this? This has to go in a header file included by both
core.c and protocol.c.
It's fixed now (I moved the declaration to http_protocol.h and
wrapped it in #ifdef CORE_PRIVATE)
--Brian