On Mon, Dec 20, 2010 at 4:11 PM, Julian Foad <julian.foad_at_wandisco.com> wrote:
> On Mon, 2010-12-20, Danny Trebbien wrote:
>> [[[
>> Add a public API function, svn_subst_translate_string2(), an extension of
>> svn_subst_translate_string(), that has two additional output parameters for
>> determining whether re-encoding and/or line ending translation were performed.
> [...]
>
> Thanks, Danny.
>
> Committed revision 1051322.
:)
> The main patch looks great. In the test file, I made the minimum
> changes necessary to comply with the C'89 language (no // comments, no
> declarations after code), and committed it. However I suggest many
> further changes to the test code, mainly making it data-driven (use a
> table of inputs and expected outputs) and use helpers such as
> SVN_TEST_STRING_ASSERT() to maximize readability. To this end, I
> suggest the version attached here. Do you want to check this for me, to
> see if it still does what you wanted it to, and let me know if I should
> commit it.
Hi Julian,
That's much better than mine, and I have verified that the tests are
the same. Please commit.
Received on 2010-12-21 22:19:27 CET