Karl Fogel wrote:
> ------------------------------------------------------------------------
> 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.
--
C. Michael Pilato <cmpilato_at_collab.net>
CollabNet <> www.collab.net <> Distributed Development On Demand
Received on 2008-05-23 05:59:40 CEST