Something like that (again just an idea). Perhaps we should really create a struct:

typedef enum {
    VOID,
    RUN_FIRST,
    RUN_ALL
} modperl_hook_run_mode_e

struct modperl_hook_t {
    const char *desc;
    modperl_hook_run_mode_e run_mode;
};

and now we can kill all the modperl_handler_desc_ functions and just return this struct from modperl_hooks.c. Later on we might fold in more information. Do you think anything else should be added there?

would those MP_ constants be helpful, such as MP_HANDLER_TYPE_PER_DIR and MP_HEADER_PARSER_HANDLER? it's hard to tell without working on an implementation.



Or is it considered a bad practice to create a struct for a static data that never changes?

I have no idea, not being a real C guy.


if we proceed this way it's probably going to take me a while to come up with another patch, since I'll need to do some reading first to fill in the gaps in my C :)

--Geoff


--------------------------------------------------------------------- To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]



Reply via email to