[email protected] wrote on Mon, Jul 22, 2013 at 16:49:25 -0000:
> Author: stefan2
> Date: Mon Jul 22 16:49:24 2013
> New Revision: 1505729
> 
> URL: http://svn.apache.org/r1505729
> Log:
> On the fsfs-improvements branch:  Rename a debug function.
> 
> * subversion/libsvn_fs_fs/cached_data.c
>   (dgb__log_access): rename to ...
>   (dbg_log_access): ... this one

That's a classic example of a log message that leaves me "Huh?"-ed: it
says nothing more than the unidiff says.  "Fix a typo in the function
name" would have been easier to understand.

This one is a trivial patch, but the point of "Describe the why, not the
what" applies to larger, less-trivial patches too.  (And that's why
I made it)

Reply via email to