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: 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 descr

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_DEC

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 wou

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: > >

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 li

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() { >

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: 44033

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=rev&rev=440337 > Log: > Replace ap_get_server_version with ap_get_server_banner() and > ap_get

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, i

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=rev&rev=440337 > Log: > Replace ap_get_server_version with ap_get_server_banner() and > ap_get_server_description(). > > High-level summary: > > T