On 17 October 2016 at 17:29, Bert Huijben <b...@qqmail.nl> wrote:
>> -----Original Message-----
>> From: i...@apache.org [mailto:i...@apache.org]
>> Sent: maandag 17 oktober 2016 15:49
>> To: comm...@subversion.apache.org
>> Subject: svn commit: r1765286 - in /subversion/trunk/subversion:
>> include/svn_xml.h libsvn_subr/xml.c tests/libsvn_subr/xml-test.c
>> Author: ivan
>> Date: Mon Oct 17 13:49:05 2016
>> New Revision: 1765286
>> URL: http://svn.apache.org/viewvc?rev=1765286&view=rev
>> Implement standard lifetime semantics for svn_xml_parser_t: the object will
>> automatically freed on pool cleanup. But it still can be freed explicitly
>> using svn_xml_free_parser(). It's the same behavior we already have for
>> svn_sqlite__db_t and similar.
> Are you planning a new use of this api?
Yes. Currently we link Expat XML three times: in libsvn_subr,
libsvn_ra_serf and mod_dontdothat. I'd like to fix it.
> It is currently only used by subversion/libsvn_wc/old-and-busted.c, and I
> think we should really spend time optimizing that specific usecase (reading
> pre 1.4 working copies)
Of course I'm aware of this.