Author: cmpilato
Date: Tue Jun 12 09:13:36 2012
New Revision: 1349203

URL: http://svn.apache.org/viewvc?rev=1349203&view=rev
Log:
Follow-up to r1348130, expanding problem space solution coverage.

* subversion/libsvn_ra_serf/util.c
  (start_xml, end_xml, cdata_xml): Use !SERF_BUCKET_READ_ERROR()
    instead of merely APR_STATUS_IS_EOF() so that all the magic serf
    error codes are wrapped.

Suggested by: rhuijben

Modified:
    subversion/trunk/subversion/libsvn_ra_serf/util.c

Modified: subversion/trunk/subversion/libsvn_ra_serf/util.c
URL: 
http://svn.apache.org/viewvc/subversion/trunk/subversion/libsvn_ra_serf/util.c?rev=1349203&r1=1349202&r2=1349203&view=diff
==============================================================================
--- subversion/trunk/subversion/libsvn_ra_serf/util.c (original)
+++ subversion/trunk/subversion/libsvn_ra_serf/util.c Tue Jun 12 09:13:36 2012
@@ -1216,7 +1216,7 @@ start_xml(void *userData, const char *ra
   svn_ra_serf__expand_ns(&name, parser->state->ns_list, raw_name);
 
   err = parser->start(parser, name, attrs, scratch_pool);
-  if (err && APR_STATUS_IS_EOF(err->apr_err))
+  if (err && !SERF_BUCKET_READ_ERROR(err->apr_err))
     err = svn_error_create(SVN_ERR_RA_SERF_WRAPPED_ERROR, err, NULL);
 
   parser->error = err;
@@ -1241,7 +1241,7 @@ end_xml(void *userData, const char *raw_
   svn_ra_serf__expand_ns(&name, parser->state->ns_list, raw_name);
 
   err = parser->end(parser, name, scratch_pool);
-  if (err && APR_STATUS_IS_EOF(err->apr_err))
+  if (err && !SERF_BUCKET_READ_ERROR(err->apr_err))
     err = svn_error_create(SVN_ERR_RA_SERF_WRAPPED_ERROR, err, NULL);
 
   parser->error = err;
@@ -1266,7 +1266,7 @@ cdata_xml(void *userData, const char *da
   scratch_pool = parser->state->pool;
 
   err = parser->cdata(parser, data, len, scratch_pool);
-  if (err && APR_STATUS_IS_EOF(err->apr_err))
+  if (err && !SERF_BUCKET_READ_ERROR(err->apr_err))
     err = svn_error_create(SVN_ERR_RA_SERF_WRAPPED_ERROR, err, NULL);
 
   parser->error = err;


Reply via email to