[PATCH] HTTP reason phrases not always returned in ra_dav error messages

While trying to diagnose whether an error a proxy error or not I noticed
not all ra_dav error messages were including the Neon error message
correctly.
 
Attached is a patch that takes a stab at fixing the problem. I'm not so
happy about what I did to props.c, any other ideas would definately be
appreciated.
 
Here's the Changelog:
 
* libsvn_ra_dav/props.c:
  (svn_ra_dav__get_props) Use error helper function to ensure
      that the HTTP reason string gets included in the error context.
* libsvn_ra_dav/util.c:
  (svn_ra_dav__parsed_request, svn_ra_dav__request_dispatch) Adjust
      the code knowing that Neon can return NE_OK while still having
      a failure status code.

FYI,
Bill

----
Do you want a dangerous fugitive staying in your flat?
No.
Well, don't upset him and he'll be a nice fugitive staying in your flat.
 
 

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