On Mon, Feb 6, 2012 at 8:50 AM, Daniel Shahaf <danie...@elego.de> wrote:
> hwri...@apache.org wrote on Mon, Feb 06, 2012 at 14:42:45 -0000:
>> +  checksum = svn_checksum_empty_checksum(svn_checksum_md5, pool);
>> +  SVN_TEST_ASSERT(svn_checksum_is_empty_checksum(checksum));
>> +
>
> That doesn't test much -- both functions just call
> svn_md5__empty_string_digest() which returns a static constant.
>
> Perhaps actually construct an empty svn_string_t and checksum it?

Makes sense: r1241033

-Hyrum



-- 

uberSVN: Apache Subversion Made Easy
http://www.uberSVN.com/

Reply via email to