[m-rev.] for review: Make some cord functions tail recursive.

Julien Fischer jfischer at opturion.com
Thu Jun 13 14:23:05 AEST 2013


On Thu, 13 Jun 2013, Peter Wang wrote:

> For review.
> ---
>
> Add test case for cords.
>
> Test the recently changed functions more comprehensively:
> cord.list, cord.rev_list, cord.cord_list_to_cord, cord.cord_list_to_list.
>
> The existing test_cord.m is deficient, and did not reveal a bug in the
> previous implementation of cord.rev_list (the non-tail-recursive one).
>
> tests/hard_coded/Mmakefile:
> tests/hard_coded/test_cord2.exp:
> tests/hard_coded/test_cord2.m:
>        Add test case.

That's fine.

Cheers,
Julien.



More information about the reviews mailing list