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

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