[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:55:23 -0400

On Thu, Jun 3, 2010 at 12:53, Greg Stein <gstein_at_gmail.com> wrote:
> 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.

(euh... that was in response to Philip's "Perhaps..." comment)
Received on 2010-06-03 18:55:58 CEST

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