Stas Bekman wrote:

> Jeff Trawick wrote:
>
> > Stas Bekman wrote:
> >
> >> Should I update Changes for this fix as well?
> >
>
> I was asking whether I should have also logged this last change:
>
>    AP_CORE_DECLARE(void) ap_process_connection(conn_rec *c, void *csd)
>    {
>   -    apr_status_t rc;
>   +    int rc;
>
> My common sense says no, but one can't be too sure...

oh, right you are... don't log something like that

if it changes code from prior release and it is user visible, log it;
otherwise, probably not if you ask me unless it is something really big like making mod_rewrite easier to read :)




Reply via email to