phpv8/php-v8

Remove excessive Isolate and Context parameters

pinepain opened this issue · 1 comments

They are often not needed (especially Isolate) and object's Context could be used instead. The Isolate and Context parameter appeared during mimicking V8 API where this parameter was largely used to deal with lack of function overriding by return type in C++, which is definitely not the case in PHP.

After taking this issue more seriously and looking into v8 internals, i don't think this change is ever necessary. It's is not trivial or feasible at all for some extension parts to implement this change. Partial extension API changes will create certain ambiguity and I don't think that it worth doing that.