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

Re: svn commit: r1501371 - /subversion/trunk/subversion/libsvn_ra_serf/util_error.c

From: Branko Čibej <brane_at_wandisco.com>
Date: Tue, 09 Jul 2013 23:12:07 +0200

On 09.07.2013 23:00, Bert Huijben wrote:
> I would welcome a patch to svn_strerror that improves the default
> error for serf specific error codes,

That would effectively make our make implementation details of
libsvn_ra_serf part of the public API. That's a non-strarter IMO.

> or a patch that translates all serf error codes to something within a
> subversion error range.

If we go down that road, we may as well make Serf part of the Subversion
project. Which I don't believe is going to happen.

By all means wrap the error messages inside libsvn_ra_serf; I wouldn't
object to extracting the Serf error message and adding a "caused-by:
serf error NNNN" in the same error chain.

-- Brane

-- 
Branko Čibej | Director of Subversion
WANdisco // Non-Stop Data
e. brane_at_wandisco.com
Received on 2013-07-09 23:12:41 CEST

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

This site is subject to the Apache Privacy Policy and the Apache Public Forum Archive Policy.