[svn.haxx.se] · SVN Dev · SVN Users · SVN Org · TSVN Dev · TSVN Users · Subclipse Dev · Subclipse Users · this month's index

Re: svn commit: r31313 - in trunk/subversion: include libsvn_client libsvn_ra_neon libsvn_ra_serf svn

From: Karl Fogel <kfogel_at_red-bean.com>
Date: Tue, 20 May 2008 14:02:37 -0400

"Eric Gillespie" <epg_at_google.com> writes:
>> Log:
>> Fix issue #3137: use a consistent error code when a path is not found in the
>> repository. DAV uses a 404 status code to indicate missing paths, so we now
>> convert that code to SVN_ERR_FS_NOT_FOUND, as we do for ra_local and ra_svn.
>
> Woohoo, thanks lgo! Any reason we can't backport this for 1.5.0?
> Shouldn't require resoak...

Agreed.

Also, if the error code is consistent now, shouldn't it be documented
where it's exposed in public APIs? The only current mention of
"SVN_ERR_FS_NOT_FOUND" in svn_ra.h is in svn_ra_get_mergeinfo()'s doc
string...

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe_at_subversion.tigris.org
For additional commands, e-mail: dev-help_at_subversion.tigris.org
Received on 2008-05-20 20:02:56 CEST

This is an archived mail posted to the Subversion Dev mailing list.