Re: svn commit: r30161 - in trunk/subversion: libsvn_ra_neon tests/cmdline

"David Glasser" <glasser_at_davidglasser.net> writes:
> I was hoping that somebody with a better understanding of our DAV
> report protocol could see if there are any other similar changes to be
> made as well first.

I think this change is spiffy (just reviewed), and you underestimate
your understanding of the DAV protocol :-).

You're right that we could open up "add-directory" to tolerate
"delete-entry" too, but I'm wary of making a change that no bug report
has called for yet. As you say, we don't send add-dir-with-history yet,
and it seems to me that when we do, this either will or won't (but
probably "will") be a problem, at which point we'll notice it. My
instinct is to leave the rest alone right now, but if we do change it,
let's keep the change in trunk and at least not backport it to 1.5
(which, in theory, would never need it).

By the way, I looked in ra_serf, and it doesn't seem to have any of
these problems. start_report() just handles whatever it sees.

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe_at_subversion.tigris.org
For additional commands, e-mail: dev-help_at_subversion.tigris.org
Received on 2008-04-04 23:41:19 CEST