You are viewing a plain text version of this content. The canonical link for it is here.
Posted to modproxy-dev@apache.org by Ian Holsman <Ia...@cnet.com> on 2001/05/30 06:19:25 UTC
[PATCH] ap_strchr* apr_strstr* ap_rind ap_ind ap_str_lower
This patch moves the following functions to apr_strings
it also #defines the old names in httpd.h for backwards
compatibility (which is intended to be removed in a month or so)
ap_ind --> apr_ind
ap_rind --> apr_rind
ap_str_tolower --> apr_str_tolower
ap_strchr_c --> apr_strchr_c
ap_strchr --> apr_strchr
ap_strrchr --> apr_strrchr
ap_strrchr_c --> apr_strrchr_c
ap_strstr --> apr_strstr
ap_strstr_c --> apr_strstr_c
there are 2 patch files
1 is for APR/httpd
the other for mod_proxy
..Ian
Re: [PATCH] ap_strchr* apr_strstr* ap_rind ap_ind ap_str_lower
Posted by Greg Stein <gs...@lyra.org>.
On Tue, May 29, 2001 at 09:19:25PM -0700, Ian Holsman wrote:
>
> This patch moves the following functions to apr_strings
> it also #defines the old names in httpd.h for backwards
> compatibility (which is intended to be removed in a month or so)
>
> ap_ind --> apr_ind
> ap_rind --> apr_rind
These should just be deleted. They're quite bogus :-)
There are three uses of ap_ind and one use of ap_rind. Not worth futzing
with those things.
>...
> --- include/httpd.h 2001/05/22 01:31:02 1.151
> +++ include/httpd.h 2001/05/30 04:00:48
>...
> +/* the following are for compatibility.
> + * These will be removed once changes are commited in APR, and
> + * all affected modules are determined
> + * TODO: REMOVE BY 2001-06-15
> + */
> +
> +#define ap_str_tolower(x) apr_str_tolower(x)
> +#define ap_ind(x,y) apr_ind(x,y)
> +#define ap_rind(x,y) apr_rind(x,y)
> +#define ap_strchr(s, c) strchr(s, c)
> +#define ap_strchr_c(s, c) strchr(s, c)
> +#define ap_strrchr(s, c) strrchr(s, c)
> +#define ap_strrchr_c(s, c) strrchr(s, c)
> +#define ap_strstr(s, c) strstr(s, c)
> +#define ap_strstr_c(s, c) strstr(s, c)
The strchr, strrchr, and strstr items should be mapped to the apr_
varieties. Otherwise, the unconverted macros lose all benefit of the const
stuff.
>...
> --- srclib/apr/include/apr_strings.h 2001/05/23 14:15:39 1.17
> +++ srclib/apr/include/apr_strings.h 2001/05/30 04:02:20
>...
> +#ifdef APR_DEBUG
This value needs to be set in apr/configure.in. Take a look at httpd's
setting of AP_DEBUG and look at the debug stuff in apr's configure.in.
Setting APR_DEBUG needs to happen, or (again) we lose the benefit of these
macros.
Basically: we shouldn't apply the patch at this point because it will simply
disable the type safety checks that the apr_strchr stuff is giving us.
Cheers,
-g
--
Greg Stein, http://www.lyra.org/
Re: [PATCH] ap_strchr* apr_strstr* ap_rind ap_ind ap_str_lower
Posted by Greg Stein <gs...@lyra.org>.
On Tue, May 29, 2001 at 09:19:25PM -0700, Ian Holsman wrote:
>
> This patch moves the following functions to apr_strings
> it also #defines the old names in httpd.h for backwards
> compatibility (which is intended to be removed in a month or so)
>
> ap_ind --> apr_ind
> ap_rind --> apr_rind
These should just be deleted. They're quite bogus :-)
There are three uses of ap_ind and one use of ap_rind. Not worth futzing
with those things.
>...
> --- include/httpd.h 2001/05/22 01:31:02 1.151
> +++ include/httpd.h 2001/05/30 04:00:48
>...
> +/* the following are for compatibility.
> + * These will be removed once changes are commited in APR, and
> + * all affected modules are determined
> + * TODO: REMOVE BY 2001-06-15
> + */
> +
> +#define ap_str_tolower(x) apr_str_tolower(x)
> +#define ap_ind(x,y) apr_ind(x,y)
> +#define ap_rind(x,y) apr_rind(x,y)
> +#define ap_strchr(s, c) strchr(s, c)
> +#define ap_strchr_c(s, c) strchr(s, c)
> +#define ap_strrchr(s, c) strrchr(s, c)
> +#define ap_strrchr_c(s, c) strrchr(s, c)
> +#define ap_strstr(s, c) strstr(s, c)
> +#define ap_strstr_c(s, c) strstr(s, c)
The strchr, strrchr, and strstr items should be mapped to the apr_
varieties. Otherwise, the unconverted macros lose all benefit of the const
stuff.
>...
> --- srclib/apr/include/apr_strings.h 2001/05/23 14:15:39 1.17
> +++ srclib/apr/include/apr_strings.h 2001/05/30 04:02:20
>...
> +#ifdef APR_DEBUG
This value needs to be set in apr/configure.in. Take a look at httpd's
setting of AP_DEBUG and look at the debug stuff in apr's configure.in.
Setting APR_DEBUG needs to happen, or (again) we lose the benefit of these
macros.
Basically: we shouldn't apply the patch at this point because it will simply
disable the type safety checks that the apr_strchr stuff is giving us.
Cheers,
-g
--
Greg Stein, http://www.lyra.org/
Re: [PATCH] ap_strchr* apr_strstr* ap_rind ap_ind ap_str_lower
Posted by Greg Stein <gs...@lyra.org>.
On Tue, May 29, 2001 at 09:19:25PM -0700, Ian Holsman wrote:
>
> This patch moves the following functions to apr_strings
> it also #defines the old names in httpd.h for backwards
> compatibility (which is intended to be removed in a month or so)
>
> ap_ind --> apr_ind
> ap_rind --> apr_rind
These should just be deleted. They're quite bogus :-)
There are three uses of ap_ind and one use of ap_rind. Not worth futzing
with those things.
>...
> --- include/httpd.h 2001/05/22 01:31:02 1.151
> +++ include/httpd.h 2001/05/30 04:00:48
>...
> +/* the following are for compatibility.
> + * These will be removed once changes are commited in APR, and
> + * all affected modules are determined
> + * TODO: REMOVE BY 2001-06-15
> + */
> +
> +#define ap_str_tolower(x) apr_str_tolower(x)
> +#define ap_ind(x,y) apr_ind(x,y)
> +#define ap_rind(x,y) apr_rind(x,y)
> +#define ap_strchr(s, c) strchr(s, c)
> +#define ap_strchr_c(s, c) strchr(s, c)
> +#define ap_strrchr(s, c) strrchr(s, c)
> +#define ap_strrchr_c(s, c) strrchr(s, c)
> +#define ap_strstr(s, c) strstr(s, c)
> +#define ap_strstr_c(s, c) strstr(s, c)
The strchr, strrchr, and strstr items should be mapped to the apr_
varieties. Otherwise, the unconverted macros lose all benefit of the const
stuff.
>...
> --- srclib/apr/include/apr_strings.h 2001/05/23 14:15:39 1.17
> +++ srclib/apr/include/apr_strings.h 2001/05/30 04:02:20
>...
> +#ifdef APR_DEBUG
This value needs to be set in apr/configure.in. Take a look at httpd's
setting of AP_DEBUG and look at the debug stuff in apr's configure.in.
Setting APR_DEBUG needs to happen, or (again) we lose the benefit of these
macros.
Basically: we shouldn't apply the patch at this point because it will simply
disable the type safety checks that the apr_strchr stuff is giving us.
Cheers,
-g
--
Greg Stein, http://www.lyra.org/