G/Evening (Is here)
A very small tweak is proposed for:
httpd-trunk\modules\http2\h2_io_set.c
to keep the C89-ers in their seats.
The svn diff looks like:
Index: modules/http2/h2_io_set.c
===
--- modules/http2/h2_io_set.c (revisio
On 08/19/2015 11:57 AM, NormW wrote:
G/Evening (Is here)
A very small tweak is proposed for:
httpd-trunk\modules\http2\h2_io_set.c
to keep the C89-ers in their seats.
thanks committed.
Cheers
Jean-Frederic
Thanks Jean-Frederic.
Is there a way to ./configure - other than CFLAGS - so that all developers can
compile with similar subsets of features and warnings? I would like to write
code so that it does not generate warnings for the rest of you...
//Stefan
> Am 19.08.2015 um 14:00 schrieb jean-fre
That's the intent for --enable-maintainer-mode
> On Aug 19, 2015, at 8:22 AM, Stefan Eissing
> wrote:
>
> Thanks Jean-Frederic.
>
> Is there a way to ./configure - other than CFLAGS - so that all developers
> can compile with similar subsets of features and warnings? I would like to
> write
> Am 19.08.2015 um 15:33 schrieb Jim Jagielski :
>
> That's the intent for --enable-maintainer-mode
Excellent. Thx!
bytes GmbH
Hafenweg 16, 48155 Münster, Germany
Phone: +49 251 2807760. Amtsgericht Münster: HRB5782
On Wed, Aug 19, 2015 at 9:13 AM, wrote:
> Author: icing
> Date: Wed Aug 19 14:13:49 2015
> New Revision: 1696592
>
> URL: http://svn.apache.org/r1696592
> Log:
> mod_h2 compiles warning free in maintainer-mode
>
> --- httpd/httpd/trunk/modules/http2/h2_alt_svc.c (original)
> +++ httpd/httpd/trunk
> Am 19.08.2015 um 16:37 schrieb William A Rowe Jr :
> h2_alt_svc *h2_alt_svc_parse(const char *s, apr_pool_t *pool) {
> -const char *sep = strchr(s, '=');
> +const char *sep = strchr((char *)s, '=');
>
>
> We solve these issues with ap_strchr_c, ap_strrchr_c, ap_strstr_c etc. This
>
Hi,
in the 2 functions below, you could also use apr_pstrcat instead of a
hard coded (20) for extra space needed by '[', ']', ... +
apr_pcalloc/strlen/strcpy/strcat
This is cleaner, IMHO and less verbose (and faster but we don't really
care here).
CJ
Le 19/08/2015 16:55, ic...@apache.org a