subversion-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Daniel Shahaf <>
Subject Re: r1592723: Add 'svnadmin delrevprop'.
Date Fri, 09 May 2014 23:50:46 GMT
Julian Foad wrote on Sat, May 10, 2014 at 00:24:34 +0100:
> Hi Daniel. It looks like your change created this warning:
> subversion/svnadmin/svnadmin.c: In function 'subcommand_delrevprop':
> subversion/svnadmin/svnadmin.c:2304:27: unused variable 'filename'

Thanks Julian.  I missed it (and so did svn-warnings).  Fixed in r1593652.

Separately, your two emails make me wonder if we should change the
commit messages' format to include the first line of the log message in
the subject line.  The repositories can
do this, and I've found that the first paragraph of the log message
usually serves well as a summary of the change for STATUS purposes.

In fact, making the log messages more prominent may encourage us to
write better summaries.


View raw message