>No! If you are going to use separate functions then write:

>static void log_access_verdict(...)
>{

It still needs a signature change to log_access_verdict.

>#if AP_MODULE_MAGIC_AT_LEAST(...)
>   log_access_verdict_httpd_v23(...)
>#else
>   log_access_verdict_httpd_v22(...)
>#endif
>}

>so that the callers don't have to change.

>On the whole I preferred the macro solution.

Me to prefer the orignal macro based solution.

With regards
Kamesh Jayachandran

Reply via email to