Re: svn commit: r1040831 - in /subversion/trunk/subversion: include/svn_checksum.h libsvn_subr/checksum.c

On 02.12.2010 12:18, Julian Foad wrote:
> A good test for whether it's worth making an API accept NULL as an input
> is: what proportion of the callers would find that useful? I see there
> are about 40 callers in the code base. Would you mind scanning through
> them and letting us know?
>
There were two places where the callers had to check
for NULL before calling svn_checksum_to_cstring().
I removed those checks in r1042460.

Two more places might try to call the function with a NULL
checksum under adverse conditions:

* 6x in dump_node() (subversion/libsvn_repos/dump.c)
   because the svn_fs_file_checksum() is not instructed
   to calculate missing checksums
* While constructing an error message in window_handler()
   (subversion\libsvn_wc\update_editor.c)

There are about ten more places where it is not entirely
crystal clear that the desired checksum is actually available.
However, I'm reasonably sure that they will in the respective
contexts (e.g. wc operations).

-- Stefan^2.
Received on 2010-12-05 23:04:28 CET