Re: cvs commit: httpd-2.0/server core.c

[Ryan Bloom <rb...@covalent.net>]

>    module AP_MODULE_DECLARE_DATA cgi_module;
>
>   +/* There has to be a better place to put this - uhm... where exactly? */
>   +/**
>   + * Reprocess the command and arguments to execute the given CGI script.
>   + * @param cmd Pointer to the command to execute (may be overridden)
>   + * @param argv Pointer to the arguments to pass (may be overridden)
>   + * @param r The current request
>   + * @param p The pool to allocate correct cmd/argv elements within.
>   + * @deffunc apr_status_t ap_cgi_build_command(const char **cmd, const
> char ***argv, request_rec *r, apr_pool_t *p) \
>   + * @tip This callback may be registered by the os-specific module 
>   + * to correct the command and arguments for apr_proc_create invocation 
>   + * on a given os.  mod_cgi will call the function if registered. 
>   + */
>   +APR_DECLARE_OPTIONAL_FN(apr_status_t, ap_cgi_build_command,
>   +                        (const char **cmd, const char ***argv,
>   +                         request_rec *r, apr_pool_t *p));
>   +

I obviously haven't had time to review this yet, but I can answer the question
above.  The APR_DECLARE_OPTIONAL_FN has to go in a header file, so that
other modules can find what it declares.  Then, the doxygen docs can go into that
header file.

Ryan

______________________________________________________________
Ryan Bloom				rbb@apache.org
Covalent Technologies			rbb@covalent.net
--------------------------------------------------------------