> ==========================================================
> ====================
> > --- subversion/trunk/subversion/svnadmin/main.c (original)
> > +++ subversion/trunk/subversion/svnadmin/main.c Sat Apr 23 13:59:23
> 2011
> > @@ -2001,4 +2001,7 @@ subcommand_crashtest(apr_getopt_t *os, v
> >
> > SVN_ERR(open_repos(&repos, opt_state->repository_path, pool));
> > abort();
> > +
> > + /* merely silence a compiler warning (this will never be executed) */
> > + return SVN_NO_ERROR;
> > }
> >
> >
>
> Heh. Now gcc emits an unreachable code error:
> subversion/svnadmin/main.c: In function ‘subcommand_crashtest’:
> subversion/svnadmin/main.c:2006: warning: will never be executed
>
> I suppose we could if-def the above line depending on the compiler.
> If we *don't* want to play such games, my heavily biased bikeshed
> would be to omit the admittedly superfluous line.
.SVN_ERR_MALFUNCTION() should work in all cases.
Bert
Received on 2011-04-24 04:00:28 CEST