"C. Michael Pilato" <cmpilato_at_collab.net> writes:
>> ------------------------------------------------------------------------
>> r31342 | cmpilato | 2008-05-21 17:24:24 -0400 (Wed, 21 May 2008) | 18 lines
>>
>> Merge from trunk this change group:
>>
>> ### I KIND OF WANT TO REVIEW THIS TOO, JUST BECAUSE IT'S SO BIG. ###
>> ### BUT IN PRINCIPLE, NO OBJECTION TO THIS BEING IN 1.5.0 (kff) ###
>>
>> * r31243, r31246, r31249, r31250, r31251, r31271
>> Fix issue #3200, API Ickiness: svn_client_ctx_t shouldn't carry
>> "extra revprops"; the svn_client APIs should
>
> This change looks much bigger than it really is. The C changes were
> of the do-it-once-then-repeat-ten-times variety. One of the changes
> is "make the variable const" type of change (the sort the compiler
> verifies for you). Then there are the bindings changes, which (again)
> mostly just make use of prior art. I bet you could review this in a
> relatively short amount of time, and the easiest thing to review would
> not be the individual original revs, but the single merged revision.
Thanks for the tip. I'd assumed it was that sort of change, yeah, but
might not have thought to do it by reviewing the single merged rev.
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe_at_subversion.tigris.org
For additional commands, e-mail: dev-help_at_subversion.tigris.org
Received on 2008-05-23 22:41:11 CEST