[Date Prev][Date Next][Thread Prev][Thread Next]
- Subject: Re: A quick methods lookup patch
- From: Mark Hamburg <mark@...>
- Date: Wed, 25 Nov 2009 11:29:52 -0800
It does, however, have the potential issue that once it goes down the __methods path at any level it won't look at __index. That may or may not be a good thing. It certainly is defensible, but I felt I should call it out. Over all, I would rate my patch as suitable for experimentation, but far from the final word on semantics and implementation.
In contrast, when I tried implementing the proposed __proxy metatable entry in luaV_gettable, it ran afoul of the need to do a recursive call to try __proxy first followed by __index. That's not fatal either, but it means we lose the nice tight loop in luaV_gettable.