Comment #2 on issue 2148 by [email protected]: Null(Isolate* isolate) crashes if we pass 0 for isolate
http://code.google.com/p/v8/issues/detail?id=2148
I don't think it's a good idea to add this to the V8 implementation of Null(Isolate*) and co. Right now, Null(Isolate*) is super fast, it's a few instructions when inlined. By adding the check always, it could make the inlined version comparatively much more expensive, even in the case that you know the isolate is not NULL. There would be no way to avoid the check. V8 also gets embedded elsewhere, not just in Chrome, and it seems like in general we want to keep the optimal path possible, since the embedder should be able to know/decided when the isolate is available or not.
-- v8-dev mailing list [email protected] http://groups.google.com/group/v8-dev
