On Mon Dec 4 12:09:11 2023 +0000, Jacek Caban wrote:
This patch gives new meaning to PRIVATE vs. NONE thunk type, unwrapping the chain of one, but not the other. Is it intentional? Currently extension chains are just always unwrapped and it matches what we need. Unless there is a reason to change it (it's not clear from this MR, the new scheme is not used anyway), I'd consider just using bool for the thunk property in FUNCTION_OVERRIDES.
It wasn't clear what it meant in this context, and there is currently no difference between `"thunk" : ThunkType.NONE` and `"thunk" : ThunkType.PRIVATE`.
I'll try to make it cleaner before changing it.