Re: svn commit: r440337 - in /httpd/httpd/trunk: ./ include/ modules/arch/netware/ modules/experimental/ modules/generators/ modules/http/ modules/mappers/ modules/proxy/ modules/ssl/ server/ server/m

2006-09-05 Thread Ruediger Pluem
On 09/05/2006 03:08 PM, wrote: Author: trawick Date: Tue Sep 5 06:08:15 2006 New Revision: 440337 URL: http://svn.apache.org/viewvc?view=revrev=440337 Log: Replace ap_get_server_version with ap_get_server_banner() and ap_get_server_description(). High-level summary: The full

Re: svn commit: r440337 - in /httpd/httpd/trunk: ./ include/ modules/arch/netware/ modules/experimental/ modules/generators/ modules/http/ modules/mappers/ modules/proxy/ modules/ssl/ server/ server/m

2006-09-05 Thread Brian Akins
Ruediger Pluem wrote: 1. If we stick to AP_DECLARE(const char *) ap_get_server_version(void); and do #define ap_get_server_banner ap_get_server_version I hate macros. Just do it like: AP_DECLARE(const char *) ap_get_server_banner() { return ap_get_server_version(); } That way,

Re: svn commit: r440337 - in /httpd/httpd/trunk: ./ include/ modules/arch/netware/ modules/experimental/ modules/generators/ modules/http/ modules/mappers/ modules/proxy/ modules/ssl/ server/ server/m

2006-09-05 Thread Jeff Trawick
On 9/5/06, Ruediger Pluem [EMAIL PROTECTED] wrote: On 09/05/2006 03:08 PM, wrote: Author: trawick Date: Tue Sep 5 06:08:15 2006 New Revision: 440337 URL: http://svn.apache.org/viewvc?view=revrev=440337 Log: Replace ap_get_server_version with ap_get_server_banner() and

Re: svn commit: r440337 - in /httpd/httpd/trunk: ./ include/ modules/arch/netware/ modules/experimental/ modules/generators/ modules/http/ modules/mappers/ modules/proxy/ modules/ssl/ server/ server/m

2006-09-05 Thread Rich
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On Tue, 05 Sep 2006 22:00:59 +0100 Jeff Trawick [EMAIL PROTECTED] wrote: On 9/5/06, Ruediger Pluem [EMAIL PROTECTED] wrote: On 09/05/2006 03:08 PM, wrote: Author: trawick Date: Tue Sep 5 06:08:15 2006 New Revision: 440337 URL:

Re: svn commit: r440337 - in /httpd/httpd/trunk: ./ include/ modules/arch/netware/ modules/experimental/ modules/generators/ modules/http/ modules/mappers/ modules/proxy/ modules/ssl/ server/ server/m

2006-09-05 Thread William A. Rowe, Jr.
Brian Akins wrote: Ruediger Pluem wrote: 1. If we stick to AP_DECLARE(const char *) ap_get_server_version(void); and do #define ap_get_server_banner ap_get_server_version I hate macros. Just do it like: AP_DECLARE(const char *) ap_get_server_banner() { return

Re: svn commit: r440337 - in /httpd/httpd/trunk: ./ include/ modules/arch/netware/ modules/experimental/ modules/generators/ modules/http/ modules/mappers/ modules/proxy/ modules/ssl/ server/ server/m

2006-09-05 Thread Ruediger Pluem
On 09/05/2006 11:49 PM, William A. Rowe, Jr. wrote: Brian Akins wrote: Ruediger Pluem wrote: 1. If we stick to AP_DECLARE(const char *) ap_get_server_version(void); and do #define ap_get_server_banner ap_get_server_version I hate macros. Just do it like: AP_DECLARE(const char *)

Re: svn commit: r440337 - in /httpd/httpd/trunk: ./ include/ modules/arch/netware/ modules/experimental/ modules/generators/ modules/http/ modules/mappers/ modules/proxy/ modules/ssl/ server/ server/m

2006-09-05 Thread Jeff Trawick
On 9/5/06, William A. Rowe, Jr. [EMAIL PROTECTED] wrote: Brian Akins wrote: Ruediger Pluem wrote: 1. If we stick to AP_DECLARE(const char *) ap_get_server_version(void); and do #define ap_get_server_banner ap_get_server_version I hate macros. Just do it like: AP_DECLARE(const char

Re: svn commit: r440337 - in /httpd/httpd/trunk: ./ include/ modules/arch/netware/ modules/experimental/ modules/generators/ modules/http/ modules/mappers/ modules/proxy/ modules/ssl/ server/ server/m

2006-09-05 Thread William A. Rowe, Jr.
Ruediger Pluem wrote: Hm, what backport are you thinking of? I thought of backporting it to 2.2.x not to 2.0.x. I see no general backwards compatibilty of modules written for 2.2.x to 2.0.x. Well, we are talking about breaking = 2.2.3 / 2.2.3 right? You are right, the new function would

Re: svn commit: r440337 - in /httpd/httpd/trunk: ./ include/ modules/arch/netware/ modules/experimental/ modules/generators/ modules/http/ modules/mappers/ modules/proxy/ modules/ssl/ server/ server/m

2006-09-05 Thread William A. Rowe, Jr.
Jeff Trawick wrote: AP_DECLARE(const char *) ap_get_server_banner() { return ap_get_server_version(); } I would prefer, if server_version goes away, and we want users to be unsurprised, that it's /* doxygen description... * @deprecated @see ap_get_server_banner */ AP_DECLARE(const