http://codereview.chromium.org/3770015/diff/1/2
File src/contexts.cc (right):

http://codereview.chromium.org/3770015/diff/1/2#newcode80
src/contexts.cc:80: Handle<Context> context(this, isolate);
On 2010/10/18 23:05:40, Dmitry Titov wrote:
For example, here, even though Context is a FixedArray, if I remove
isolate
parameter, the compiler will generate a constructor with
Isolate::Current().

It does it even though the types are obvious here. This happens in
multiple
other places as well - see, for example a AssertNoContextChange
constructor,
where it tries to construct another Handle<Context>...

Right, my bad. I just fixed this.

http://codereview.chromium.org/3770015/show

--
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev

Reply via email to