Re: [PATCH] (alternate fix for issue 900)

mark benedetto king <bking@Inquira.Com> writes:

> What I'm getting at is that svn_repos_open can't tell when
> to return that error, either, without peeking inside the
> error in *exactly the same way* that we're describing as
> inappriate for svn_ra_local__split_URL to do.
>
> Am I missing something?

It's not exactly the same, svn_repos_open would not need to follow the
error chain. It could just check err->apr_err, which is how all the
other error checks work. Doing things differently inevitably raises
questions. That doesn't mean your patch is wrong :)

-- 
Philip Martin
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org