Re: svn commit: r21016 - in trunk: . subversion/include/private subversion/libsvn_fs_base/bdb subversion/libsvn_subr

On Wed, 09 Aug 2006, Philip Martin wrote:

> rooneg@tigris.org writes:
>
> > Author: rooneg
> > Date: Tue Aug 8 08:34:59 2006
> > New Revision: 21016
>
> > + * Call an initialization function in a thread-safe manner.
> > + *
> > + * @a global_status must be a pointer to a global, zero-initialized
> > + * #svn_atomic_t. @a init_func is a pointer to the function that performs
> > + * the actual initialization.
> > + *
> > + * @since New in 1.5.
> > + */
> > +svn_error_t *
> > +svn_atomic_init_once(volatile svn_atomic_t *global_status,
> > + svn_error_t *(*init_func)(void));
>
> Will this always be used in places where no parameters need to be
> passed? How about:
>
> svn_error_t *
> svn_atomic_init_once(volatile svn_atomic_t *global_status,
> svn_error_t *(*init_func)(void *baton),
> void *baton);

I like Philip's suggestion -- it's a more forward-looking API.