Bill Coffman <[EMAIL PROTECTED]> wrote:
> Hello,

> I have addressed almost all the code issues from the last patch.

> * I have the below failed tests.

> Failed Test        Stat Wstat Total Fail  Failed  List of Failed
> -------------------------------------------------------------------------------
> t/library/dumper.t   13  3328    13   13 100.00%  1-13
> t/op/gc.t             1   256    18    1   5.56%  13
> t/pmc/sub.t           1   256    78    1   1.28%  78

These tests should be sane. While the first two .t are fairly complex,
t/pmc/sub_78.imc is quite simple. You could compare the produced code
of "parrot -o- foo.imc". Run it with "-t" to see, where output differs.

> * Still have a memory leak that blocks Dan's evil code from compiling.

$ valgrind --leak-check=yes --show-reachable=yes ./parrot --leak-test foo.imc

(there are some other leaks, so just have a look at increasing leak
counts for increased symbol usage)

> * Run time isn't so great,

We'll address that later.

Please post patch.

leo

Reply via email to