Will Coleda <[EMAIL PROTECTED]> wrote: > Attached is a patch that allows "(cd languages/tcl && make test)" to work > again. Looks like an unguarded access in MRO. Didn't self apply so Leo could > poke at it, in case the tcl pmcs are exploiting a hole in pmc2c2.pl.
The C<TclObject> is defined as an abstract class, i.e. it has no VTABLE. Per convention, such abstract classes should be lower case. Is there a specific reason that it's an abstract PMC? I tried to get rid of "abstract noinit" in that PMC, which produced these failing tests: cmd_break.t 1 256 2 1 50.00% 2 cmd_continue.t 1 256 2 1 50.00% 2 cmd_linsert.t 4 1024 5 4 80.00% 1 3-5 cmd_string.t 6 1536 35 6 17.14% 26-27 29-31 34 leo
