httpd-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Stefan Fritsch ...@sfritsch.de>
Subject Re: svn commit: r1210378 - /httpd/httpd/trunk/server/util_expr_eval.c
Date Tue, 13 Dec 2011 05:57:05 GMT
On Tuesday 13 December 2011, Guenter Knauf wrote:
> Hi Stefan,
> 
> Am 05.12.2011 10:38, schrieb sf@apache.org:
> > Author: sf
> > Date: Mon Dec  5 09:38:44 2011
> > New Revision: 1210378
> > 
> > URL: http://svn.apache.org/viewvc?rev=1210378&view=rev
> > Log:
> > Fix a few compiler warning reported by Steffen:
> > - some signed/unsigned mismatches
> > - const for a function does not make sense
> > 
> > Modified:
> >      httpd/httpd/trunk/server/util_expr_eval.c



> I got reported that since this commit NetWare compilation breaks:
>  > CC   server/util_expr_eval.c
>  > ### mwccnlm Compiler:
>  > #    File: server\util_expr_eval.c
>  > # --------------------------------
>  > #      92:          result = ap_expr_eval_var(ctx,
>  > node->node_arg1,
> 
> node->node_arg2);
> 
>  > #   Error:
>                ^
> 
>  > #   illegal implicit conversion from 'const void *const ' to
>  > #   'char * (*)(struct  *, const void *)'
>  > ### mwccnlm Compiler:
>  > #     677:      ap_expr_op_unary_t *op_func = info->node_arg1;
>  > #   Error:                                                   ^
>  > #   illegal implicit conversion from 'const void *const ' to
>  > #   'int (*)(struct  *, const void *, const char *)'
>  > ### mwccnlm Compiler:
>  > #     690:      ap_expr_op_binary_t *op_func = info->node_arg1;
>  > #   Error:                                                    ^
>  > #   illegal implicit conversion from 'const void *const ' to
>  > #   'int (*)(struct  *, const void *, const char *, const char
>  > *)'
> 
> sorry, but I only post due to RC1 plans in the hope you can take a
> look - I had absolutely no time yet to look at (and ATM still dont
> have) ....
> 
> greets, Gün.

I think r1213567/r1213570 should fix it. Would be nice if you (or 
someone else) could check it, though.

Cheers,
Stefan

Mime
View raw message