RE: svn commit: r1627333 - in /subversion/trunk/subversion/libsvn_ra_serf: blame.c commit.c get_lock.c lock.c log.c merge.c ra_serf.h xml.c

> -----Original Message-----
> From: Branko Čibej [mailto:brane_at_wandisco.com]
> Sent: donderdag 25 september 2014 09:30
> To: Subversion Development
> Subject: Re: svn commit: r1627333 - in
> /subversion/trunk/subversion/libsvn_ra_serf: blame.c commit.c get_lock.c
> lock.c log.c merge.c ra_serf.h xml.c
>
> On 24.09.2014 16:25, rhuijben_at_apache.org wrote:
> > Author: rhuijben
> > Date: Wed Sep 24 14:25:38 2014
> > New Revision: 1627333
> >
> > URL: http://svn.apache.org/r1627333
> > Log:
> > Tweak ra_serf report xml generation code a tiny bit to use the shorter
> > '<element />' notation for elements that are always just markers for
> > enabling a feature instead of '<element></element>'.
>
> You're aware that, depending on the DAV schema definition, it may not
> always be valid to convert an empty element to a void element, right?

Sure,

But this is simply returning to the old neon form of the requests, and for the DAV part following how they are in most example code in the RFCs.
(Slightly shrinking the requests and avoiding a very tiny bit of processing on the server and client side...)

This is also the reason I didn't just implement using the short form in svn_ra_serf__add_tag_buckets().

        Bert
Received on 2014-09-25 11:20:59 CEST