On Tue, May 17, 2011 at 10:46, Daniel Shahaf <d...@daniel.shahaf.name> wrote:
> gst...@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.

> Could you add a docstring please?  It seems that you call TXN_CB within
> a txn, then call WORK_CB (odd name IMO) with NOTIFY_FUNC, then run
> the FINALIZE_IDXth statement...

Sure.

Cheers,
-g

Reply via email to