On Sunday 17 February 2008 07:01:00 Paul Cochrane wrote:

> the attached patch removes the instantiate opcode (see RT#48022).  The
> patch also removes three tests in t/pmc/integer.t, which is something
> I'm not 100% sure about.  After I'd tried to update the tests to use
> "new" instead of "instantiate" (but without total success), and after
> an off-list discussion with kjs++, I got the feeling that the three
> tests were basically testing the "instantiate" opcode and so since the
> opcode is no longer there, that the tests should be removed.  However,
> since I'm somewhat loath to remove tests it was decided it would be
> best to ask the list.  So, yeah, what are people's opinions?

+1

-- c

Reply via email to