On Sat, May 9, 2015 at 5:58 AM, Nick Wellnhofer <[email protected]> wrote: > In the commit I mentioned, I removed the `parcel` field from `CFCSymbol` and > `S_lazy_init_sig` can't call `CFCMethod_get_parcel` anymore.
Ah, now I get it. :) > We could pass the parcel directly but I opted to pass the current class > which also knows about the parcel. I think it's easier to understand if we pass in the parcel. Thinking that the specific invoker class is somehow relevant threw me off. Also, if we continue to cache, we should keep a reference to the parcel (or capture its name), and check on subsequent calls that the cached version is valid. If you agree with these modificationss, we can either make them now or in a new commit after you finish the big merge. I'm wary of cascading changes when rebasing large branches with lots of conflicts. Marvin Humphrey
