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)".