Re: svn commit: r38350 - trunk/subversion/libsvn_ra_serf

On Mon, Jul 6, 2009 at 19:57, C. Michael Pilato<cmpilato_at_collab.net> wrote:
>...
> +++ trunk/subversion/libsvn_ra_serf/locks.c     Mon Jul  6 10:57:58 2009        (r38350)
>...
> @@ -411,7 +410,6 @@ handle_lock(serf_request_t *request,
>                                     _("Lock request failed: %d %s"),
>                                     ctx->status_code, ctx->reason));
>         }
> -
>       SVN_ERR(err);
>     }

Looks like you missed another chance for a simple "return err;" here.

Then, the following "else" clause can then go away, and the "return
SVN_NO_ERROR;" at the very end of the function can go away.

Cheers,
-g

------------------------------------------------------
http://subversion.tigris.org/ds/viewMessage.do?dsForumId=462&dsMessageId=2368462
Received on 2009-07-06 20:05:32 CEST