On 04/18/2012 03:35 PM, Greg Stein wrote:
> On Wed, Apr 18, 2012 at 18:11,<blair_at_apache.org> wrote:
>> Author: blair
>> Date: Wed Apr 18 22:11:43 2012
>> New Revision: 1327703
>>
>> URL: http://svn.apache.org/viewvc?rev=1327703&view=rev
>> Log:
>> Add and use svn_checksum__from_digest_sha1().
>>
>> This replaces svn_checksum__from_digest() taking a svn_checksum_sha1
>> argument with a new svn_checksum_t constructor function.
>>
>> * subversion/include/private/svn_subr_private.h,
>> * subversion/libsvn_subr/checksum.c
>> (svn_checksum__from_digest_sha1):
>> New private function.
>>
>> * subversion/libsvn_fs_base/util/fs_skels.c
>> (svn_fs_base__parse_representation_skel):
>> Replace
>> svn_checksum__from_digest(digest, svn_checksum_sha1, pool);
>> with
>> svn_checksum__from_digest_sha1(digest, pool);
>
> Shouldn't we be using svn_checksum_serialize() and _deserialize()
> rather than raw digests?
>
> (iow, we "shouldn't" need from_digest_sha1)
They don't contain the same bytes. I wasn't aware of
svn_checksum_serialize() till now, but those prepend a checksum type
string to the hex representation, as these are raw digest bytes.
This work was to get rid of svn_checksum__from_digest(), which is now
almost complete, per the other thread.
Blair
Received on 2012-04-19 00:42:56 CEST