httpd-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Gregg Smith <...@gknw.net>
Subject Re: svn commit: r1420377 - in /httpd/httpd/trunk: docs/manual/mod/mod_lua.xml modules/lua/lua_apr.c modules/lua/lua_apr.h modules/lua/mod_lua.c
Date Sat, 15 Dec 2012 20:41:13 GMT
Hi Daniel,
On 12/14/2012 12:46 PM, Daniel Gruno wrote:
> Bloody Windows ;)

I know, but what can you do? :)

> I have added an #ifdef for the tms stuff, as I saw mod_status did the
> same. If there is another way of making it work, I will change it as
> soon as I find a proper way to deal with it.
All these worked so at least it will compile now.

I was pointed to this one by a Netware guy asking if I'm seeing it too, 
and I am;
c:\build4\httpd-head-r1422296\modules\lua\lua_apr.c(430) : warning 
C4700: uninitialized local variable 'err'

> CC   lua_apr.c
> ### mwccnlm Compiler:
> #    File: lua_apr.c
> # ------------------
> #     412:      const char *pattern, *source, *err;
> # Warning:                                    ^
> #   variable 'err' is not initialized before being used 

Two other things I noticed now that some of the noise has settled are;
c:\build4\httpd-head-r1422296\modules\lua\lua_apr.c(1125) : warning 
C4716: 'ap_lua_load_httpd_functions' : must return a value
c:\build4\httpd-head-r1422296\modules\lua\mod_lua.c(120) : warning 
C4715: 'scope_to_string' : not all control paths return a value

On mod_lua.c look at your first two cases at lines 104 & 105, is that 
correct?

Cheers,
Gregg


Mime
View raw message