Evgeny Kotkov wrote on Thu, May 21, 2015 at 18:23:22 +0300:
> @@ -2464,24 +2452,26 @@ svn_repos_verify_fs3(svn_repos_t *repos,
>                                    cancel_func, cancel_baton,
>                                    iterpool);
>  
> +        if (err && err->apr_err == SVN_ERR_CANCELLED)
> +        ...
> +        else if (err && !keep_going)
> +        ...
> +        else if (err && keep_going)
> +        ...
> +        else if (!err && notify_func)

The "!err &&" part of last condition is tautologic.  I'd remove it,
leaving the condition as just "[else] if (notify_func)".

Reply via email to