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

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

From: Bill Tutt <rassilon_at_lyra.org>
Date: 2002-12-20 01:04:20 CET

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

Received on Fri Dec 20 01:05:09 2002

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