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

Re: svn commit: r1101738 - in /subversion/trunk/subversion/libsvn_wc: wc-queries.sql wc_db.c

From: Greg Stein <gstein_at_gmail.com>
Date: Tue, 17 May 2011 11:41:34 -0400

On Tue, May 17, 2011 at 11:29, Greg Stein <gstein_at_gmail.com> wrote:
> On Tue, May 17, 2011 at 11:26, Daniel Shahaf <d.s_at_daniel.shahaf.name> wrote:
>> Greg Stein wrote on Tue, May 17, 2011 at 11:19:40 -0400:
>>> On Tue, May 17, 2011 at 10:46, Daniel Shahaf <d.s_at_daniel.shahaf.name> wrote:
>>> > gstein_at_apache.org wrote on Wed, May 11, 2011 at 04:06:57 -0000:
>>> >> +static svn_error_t *
>>> >> +with_finalization(void *baton,
>>> >
>>> > @a BATON is unused.
>>>
>>> Yeah. I put it in there simply based on precedent and our consistent
>>> pattern for stuff like this. Nobody seems to need or use it yet.
>>>
>>
>> I'm not particularly surprised that no one uses a parameter which the
>> function's body (definition) doesn't use.
>>
>> In other words: @a BATON is not paired with any callback type
>> (pointer-to-function type), so how would it possibly be used?
>
> Oh. Wait. I was thinking of work_cb. Nobody uses that (so far).
>
> Not sure what BATON was for. I think at one point, with_finalization
> was going to fit some other signature. I'll fix it when I add the
> docstring. Thanks!

r1104342
Received on 2011-05-17 17:42:03 CEST

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

This site is subject to the Apache Privacy Policy and the Apache Public Forum Archive Policy.