On Mon, Nov 24, 2014 at 06:12:57AM -0000, Father Chrysostomos wrote:
> Thank you. I have applied both patches, as b55b14d0f and e3384dcea.
Oleg, note that B.xs needs updating to provide B::METHOP::op_class_sv
and B::METHOP::op_class_targ methods.
Although that may be unnecessary if this pans out:
> However, I wonder whether we couldn't simplify things even more. The> stash name argument (from the left-hand side of ->) is still pushed> on to the stack. If we just make *that* into a shared-hash scalar at> compile time, would it not speed things up just as much, but without> having to make methops bigger? As it is now, we end up with two SVs> in the op tree for the same class name, whereas I think we could> use just one.
--
If life gives you lemons, you'll probably develop a citric acid allergy.