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

Re: [PATCH] Issue #660: Add basic support for redirects to ra_dav

From: Daniel Rall <dlr_at_collab.net>
Date: 2006-09-11 23:07:17 CEST

On Sat, 19 Aug 2006, Daniel Westermann-Clark wrote:

> Apologies for the slow response. I've attempted to take your
> suggestions into account, but the changes are a bit more significant
> this time around. :)

Very nice job, Daniel! From your response to the review, to the log
message, to change itself, fantastic work all around.

> On 2006-08-01 10:07:46 -0700, Daniel Rall wrote:
> > commit.c has the interrogate_for_location() routine for doing this
> > work. Would it be possible to refactor both variations of
> > "Location" header retreival logic into there, and maintain an API
> > which is compliant with Neon 0.25?
>
> I've done a bit of refactoring to svn_ra_dav__request_interrogator
> in order to make it backwards compatible.
>
> The function interrogate_for_location from commit.c has been moved to
> util.c.
>
> The two request interrogator arguments to svn_ra_dav__request_dispatch
> are now part of the function's prototype regardless of Neon version.
> This required making the function interrogate_for_content_type in
> fetch.c backwards compatible also.
>
> > While we don't use ne_free() in other places we access the
> > "Location" header, since we get the data from Neon I think we
> > should. (It's simply #define'd to free() on my Linux box.)
>
> I was unsure about this; I've changed it to ne_free in the latest
> version of the patch.
...

I've made some very minor tweaks, and have committed your patch to
Subversion's trunk in r21442.

Thanks!

- Dan

  • application/pgp-signature attachment: stored
Received on Mon Sep 11 23:09:49 2006

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.