[svn.haxx.se] · SVN Dev · SVN Users · SVN Org · TSVN Dev · TSVN Users · Subclipse Dev · Subclipse Users · this month's index

Re: svn commit: r949964 - /subversion/trunk/subversion/libsvn_wc/wc_db.c

From: Greg Stein <gstein_at_gmail.com>
Date: Thu, 3 Jun 2010 12:53:41 -0400

On Tue, Jun 1, 2010 at 08:40, Philip Martin <philip.martin_at_wandisco.com> wrote:
> "Hyrum K. Wright" <hyrum_wright_at_mail.utexas.edu> writes:
>
>>>  scratch_pool);
>>> -      const char *properties = svn_sqlite__column_text(stmt, 6,
>>> scratch_pool);
>>> +      apr_size_t props_size;
>>> +      const char *properties = svn_sqlite__column_blob(stmt, 6,
>>> &props_size,
>>> +                                                       scratch_pool);
>>>
>>
>> Should we be using svn_sqlite__column_properties() here (and extending
>> _bindf() so that it knows about properties)?  I understand we're just doing
>> a straight copy, so we don't necessarily need to parse and unparse the
>> properties, but if this is the case, perhaps a comment in the code would be
>> useful.
>

It is still nice to have a comment that the properties are being
passed as an unserialized blob. That code will live for another couple
months (probably), and who knows how many times it will be read (or
copied or referred to as an example) in the meantime.

Cheers,
-g
Received on 2010-06-03 18:54:18 CEST

This is an archived mail posted to the Subversion Dev mailing list.