Re: cvs commit: parrot/ops pmc.ops

2004-12-16 Thread Leopold Toetsch
Sam Ruby wrote: Before this line is executed, (gdb) p interpreter->ctx.bp $1 = (struct parrot_regs_t *) 0x40b6bd88 After the above line is executed: (gdb) p interpreter->ctx.bp $2 = (struct parrot_regs_t *) 0x40b6bae8 Then is obviously your implementation of get_repr broken. Two questions c

Re: cvs commit: parrot/ops pmc.ops

2004-12-16 Thread Sam Ruby
Leopold Toetsch wrote: Sam Ruby wrote: Leopold Toetsch wrote: +inline op get_repr(out STR, in PMC) { +STRING *s = $2->vtable->get_repr(interpreter, $2); +$1 = s; goto NEXT(); } Strange. Stranger. Strangest. Did the test fail with JIT/i386 only? I didn't mean to commit the "inline

Re: cvs commit: parrot/ops pmc.ops

2004-12-16 Thread Leopold Toetsch
Sam Ruby wrote: Leopold Toetsch wrote: +inline op get_repr(out STR, in PMC) { +STRING *s = $2->vtable->get_repr(interpreter, $2); +$1 = s; goto NEXT(); } Strange. Stranger. Strangest. Did the test fail with JIT/i386 only? I didn't mean to commit the "inline" portion of the chan

Re: cvs commit: parrot/ops pmc.ops

2004-12-16 Thread Sam Ruby
Leopold Toetsch wrote: Sam Ruby <[EMAIL PROTECTED]> wrote: I don't understand this. At all. But the test case added to pyclass.t (motivated by test 4 in pie/b3.t) only passes if this change to the get_repr op is made. [ ... ] -op get_repr(out STR, in PMC) { -$1 = $2->vtable->get_repr(int

Re: cvs commit: parrot/ops pmc.ops

2004-12-15 Thread Leopold Toetsch
Sam Ruby <[EMAIL PROTECTED]> wrote: > I don't understand this. At all. But the test case added to pyclass.t > (motivated by test 4 in pie/b3.t) only passes if this change to the > get_repr op is made. [ ... ] > -op get_repr(out STR, in PMC) { > -$1 = $2->vtable->get_repr(interpret