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

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

From: Branko Čibej <brane_at_wandisco.com>
Date: Thu, 25 Sep 2014 11:47:10 +0200

On 25.09.2014 11:20, Bert Huijben wrote:
>
>> -----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().

Ack, thanks.

-- Brane
Received on 2014-09-25 11:47:45 CEST

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