Greg Stein <gstein_at_gmail.com> writes:
> Yes, svn_sqlite__update() *does* reset the statement before returning
> (it isn't something where you can iterate over results; it is done; so
> it resets the statement).
>
> Thus, the extra reset should not be there. "Following the pattern"
> isn't right here.
Ah, I didn't understand Bert's point. So we need a NULL.
>
>
> On Fri, Jul 16, 2010 at 13:44, Philip Martin <philip.martin_at_wandisco.com> wrote:
>> Bert Huijben <bert_at_qqmail.nl> writes:
>>
>>> Doesn't _update handle this specific reset? If not I think it should.
>>
>> The compiler was complaing about a missing argument to
>> svn_error_createf. Â I was just following the pattern used by next
>> function.
>>
>>> Â Â if (affected_rows != 1)
>>> Â Â Â return svn_error_createf(SVN_ERR_WC_PATH_NOT_FOUND,
>>> + Â Â Â Â Â Â Â Â Â Â Â Â Â Â svn_sqlite__reset(stmt),
>>> Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â _("The node '%s' was not found."),
>>> Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â svn_dirent_local_style(local_abspath,
>>> Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â scratch_pool));
>>
>> --
>> Philip
>>
>
--
Philip
Received on 2010-07-16 20:30:07 CEST