You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@subversion.apache.org by Branko Čibej <br...@wandisco.com> on 2014/09/25 09:29:53 UTC
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@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?
-- Brane
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
Posted by Branko Čibej <br...@wandisco.com>.
On 25.09.2014 11:20, Bert Huijben wrote:
>
>> -----Original Message-----
>> From: Branko Čibej [mailto:brane@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@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
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
Posted by Bert Huijben <be...@qqmail.nl>.
> -----Original Message-----
> From: Branko Čibej [mailto:brane@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@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