[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