Re: svn commit: r1656893 - in /subversion/trunk/subversion: svnsync/sync.c tests/cmdline/svnsync_tests_data/mergeinfo-contains-r0.dump tests/cmdline/svnsync_tests_data/mergeinfo-contains-r0.expected.dump
From: Julian Foad <julianfoad_at_btopenworld.com>
Date: Thu, 5 Feb 2015 13:05:25 +0000
Ivan Zhakov wrote:
> On 4 February 2015 at 20:45, Julian Foad wrote:
The parser contains checks including a check that the start rev is not zero.
I would have to split the low-level parsing from the validation feature of the current parser.
I started doing this... but didn't know when to stop. (There are lots of things the current parser currently checks for, and canonicalizes, and so lots of possibilities for separating low-level parsing from higher-level processing.)
I then thought writing the stripping-r0 code as a text manipulation would be "simpler" in the sense of a smaller change.
> 2. For every path remove zero revision using svn_rangelist_remove()
So, yes, this is how it *should* be done, I agree. Maybe I will do it.
This is an archived mail posted to the Subversion Dev mailing list.