Re: svn commit: r1702948 - in /httpd/httpd/trunk: acinclude.m4 configure.in include/http_config.h include/scoreboard.h modules/ssl/ssl_engine_kernel.c server/util_expr_eval.c

[Yann Ylavic]

On Mon, Sep 14, 2015 at 3:29 PM,  <ic...@apache.org> wrote:
> Author: icing
> Date: Mon Sep 14 13:29:35 2015
> New Revision: 1702948
>
> URL: http://svn.apache.org/r1702948
> Log:
> httpd compiles warning free on gcc and every new warning will be treated as 
> an error, standard c-89 is enforced
>
> Modified:
[]
>     httpd/httpd/trunk/server/util_expr_eval.c
>
[]
> Modified: httpd/httpd/trunk/server/util_expr_eval.c
> URL: 
> http://svn.apache.org/viewvc/httpd/httpd/trunk/server/util_expr_eval.c?rev=1702948&r1=1702947&r2=1702948&view=diff
> ==============================================================================
> --- httpd/httpd/trunk/server/util_expr_eval.c (original)
> +++ httpd/httpd/trunk/server/util_expr_eval.c Mon Sep 14 13:29:35 2015
> @@ -98,7 +98,7 @@ static const char *ap_expr_eval_word(ap_
>      case op_String:
>          result = node->node_arg1;
>          break;
> -    case op_Var:
> +

This "case" removal looks incorrect (already mentionned by Rüdiger, IIRC).

>          result = ap_expr_eval_var(ctx, (ap_expr_var_func_t *)node->node_arg1,
>                                    node->node_arg2);
>          break;

Regards,
Yann.