RE: svn commit: r1505006 - in /subversion/trunk: ./ subversion/include/svn_io.h subversion/libsvn_client/patch.c subversion/libsvn_fs_fs/fs_fs.c subversion/libsvn_subr/io.c
From: Bert Huijben <bert_at_qqmail.nl>
Date: Sat, 20 Jul 2013 00:05:03 +0200
> -----Original Message-----
Can you combine the original log messages to get a proper description here with the function additions. This makes it much easier to see when code was introduced when looking through the file history.
Looking at just this merge it is not clear why svn_io_file_create_binary() needs to be added, and what additional features these functions have over svn_io_file_create(), which already creates a text file with binary encoding.
With the diff it is clear, but the reason why belongs in the log message.
Looking at the diff, I understand the value of svn_io_file_create_binary(), but svn_io_file_create_empty() still looks like a convenience function for creating empty files, while you could just pass "", 0 to svn_io_file_create_binary().
And looking at the performance of the old working copy format on Windows, I'm certainly not recommending new code to create many empty files at once.
This is an archived mail posted to the Subversion Dev mailing list.