APR_HOOK_LINK used for optional hook request_status in mod_proxy

2011-04-03 Thread Rainer Jung
There is APR_HOOK_LINK(request_status) in mod_proxy.c. The request_status hook is an optional hook and I have the impression that this macro should only be used for non-optional hooks? The macro publishes the apr_array_header_t for the hook, but optional hooks don't use those. They use

Re: svn commit: r1086756 - in /httpd/httpd/trunk: docs/manual/developer/new_api_2_4.xml include/ap_mmn.h include/http_config.h modules/lua/mod_lua.c server/config.c server/util.c

2011-04-03 Thread Ruediger Pluem
On 03/29/2011 11:29 PM, s...@apache.org wrote: Author: sf Date: Tue Mar 29 21:29:34 2011 New Revision: 1086756 URL: http://svn.apache.org/viewvc?rev=1086756view=rev Log: Change the ap_cfg_getline() and ap_cfg_getc() to return an error code. Also: - Make ap_cfg_getline() return

Re: svn commit: r1087862 - /httpd/httpd/trunk/modules/slotmem/mod_slotmem_shm.c

2011-04-03 Thread Ruediger Pluem
On 04/01/2011 09:25 PM, j...@apache.org wrote: Author: jim Date: Fri Apr 1 19:25:19 2011 New Revision: 1087862 URL: http://svn.apache.org/viewvc?rev=1087862view=rev Log: More cleanups at cleanup... move to using pconf and then remove any file-based shms Modified:

Re: svn commit: r1086756 - in /httpd/httpd/trunk: docs/manual/developer/new_api_2_4.xml include/ap_mmn.h include/http_config.h modules/lua/mod_lua.c server/config.c server/util.c

2011-04-03 Thread Stefan Fritsch
On Sun, 3 Apr 2011, Ruediger Pluem wrote: On 03/29/2011 11:29 PM, s...@apache.org wrote: Author: sf Date: Tue Mar 29 21:29:34 2011 New Revision: 1086756 URL: http://svn.apache.org/viewvc?rev=1086756view=rev Log: Change the ap_cfg_getline() and ap_cfg_getc() to return an error code. Also: -