cacknin@tigris.org writes:
> Author: cacknin
> Date: Sat Sep 8 11:00:26 2007
> New Revision: 26498
>
> Log:
> * subversion/libsvn_wc/diff.c
> (transmit_svndiff, close_file, change_file_prop, change_dir_prop):
> remove unused variables.
>
> * subversion/libsvn_client/repos_diff.c
> (apply_textdelta): remove unused variables.
>
> * subversion/libsvn_client/patch.c
> (get_empty_file): fix messy use of ||
> (merge_file_added): avoid useless sprintf()
> (open_directory, add_file, open_file, apply_textdelta,
> change_file_prop, change_dir_prop): remove unused variables.
>
> * subversion/libsvn_client/diff.c
> (diff_dir_added, diff_dir_deleted): remove unused variables.
>
> Messy || and useless sprintf()
> Found by: glasser
It's better to put the summary line at the start of the log message,
not the end. That way, the reader can already be in the proper frame
of mind when reading the detailed part of the log message.
-K
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Tue Sep 11 20:25:56 2007