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

Re: svn commit: r1616338 - /subversion/trunk/subversion/libsvn_fs_fs/fs.c

From: Stefan Sperling <stsp_at_elego.de>
Date: Mon, 11 Aug 2014 18:41:36 +0200

On Mon, Aug 11, 2014 at 06:35:09PM +0200, Stefan Fuhrmann wrote:
> On Wed, Aug 6, 2014 at 10:59 PM, Stefan Sperling <stsp_at_elego.de> wrote:
> > > @@ -245,6 +245,16 @@ initialize_fs_struct(svn_fs_t *fs)
> > > return SVN_NO_ERROR;
> > > }
> > >
> > > +/* Reset vtable and fsap_data fields in FS such that the FS is basically
> > > + * closed now. Note that FS must not hold locks when you call this. */
> > > +static svn_error_t *
> > > +uninitialize_fs_struct(svn_fs_t *fs)
> >
> > I'd suggest to declare this as:
> >
> > static void
> > uninitialize_fs_struct(svn_fs_t *fs)
> >
>
> Thanks for the review.
>
> You are right, I might change that to plain void but I'll keep
> the error return type for now, just for symmetry with the init
> code. This is also a function that might have actual error
> conditions in later releases and a missed caller update would
> than create an error leak (and probably worse).
>
> -- Stefan^2.

By that logic we shouldn't have any void functions ;-)

I'm still in favour of changing it because it makes the code look
untidy from my point of view. But whatever...
Received on 2014-08-11 18:42:06 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.