https://codereview.chromium.org/196103004/diff/40001/src/api.cc
File src/api.cc (right):
https://codereview.chromium.org/196103004/diff/40001/src/api.cc#newcode6154
src/api.cc:6154: i::Handle<i::String> part =
i::handle(i_isolate->heap()->for_string());
On 2014/03/24 15:17:19, Michael Starzinger wrote:
nit: Just use i_isolate->factory()->for_string() instead.
Done.
https://codereview.chromium.org/196103004/diff/40001/src/api.cc#newcode6172
src/api.cc:6172: i::Handle<i::String> part =
i::handle(i_isolate->heap()->for_api_string());
On 2014/03/24 15:17:19, Michael Starzinger wrote:
nit: Just use i_isolate->factory()->for_api_string() instead.
Done.
https://codereview.chromium.org/196103004/diff/40001/src/api.cc#newcode6203
src/api.cc:6203: i::handle(i_isolate->heap()->private_api_string());
On 2014/03/24 15:17:19, Michael Starzinger wrote:
nit: Just use i_isolate->factory()->private_api_string() instead.
Done.
https://codereview.chromium.org/196103004/diff/40001/src/isolate.cc
File src/isolate.cc (right):
https://codereview.chromium.org/196103004/diff/40001/src/isolate.cc#newcode2348
src/isolate.cc:2348: return
handle(JSObject::cast(heap()->symbol_registry()));
On 2014/03/24 15:17:19, Michael Starzinger wrote:
nit: Better use "return
Handle<JSObject>::cast(factory()->symbol_registry())".
Done.
https://codereview.chromium.org/196103004/diff/40001/src/isolate.h
File src/isolate.h (right):
https://codereview.chromium.org/196103004/diff/40001/src/isolate.h#newcode1129
src/isolate.h:1129: Handle<JSObject> GetSymbolRegistry();
On 2014/03/24 15:17:19, Michael Starzinger wrote:
nit: Can we add a short one-line comment of the intended use?
Done.
https://codereview.chromium.org/196103004/diff/40001/src/runtime.cc
File src/runtime.cc (right):
https://codereview.chromium.org/196103004/diff/40001/src/runtime.cc#newcode629
src/runtime.cc:629: Handle<String> part =
handle(isolate->heap()->private_intern_string());
On 2014/03/24 15:17:19, Michael Starzinger wrote:
nit: You can just use isolate->factory()->private_intern_string()
here.
Done.
https://codereview.chromium.org/196103004/diff/40001/src/runtime.cc#newcode659
src/runtime.cc:659: HandleScope hs(isolate);
On 2014/03/24 15:17:19, Michael Starzinger wrote:
nit: s/hs/scope/ for consistency and grep-ability.
Done.
https://codereview.chromium.org/196103004/diff/40001/src/symbol.js
File src/symbol.js (right):
https://codereview.chromium.org/196103004/diff/40001/src/symbol.js#newcode67
src/symbol.js:67: var registry = %SymbolRegistry().for_intern;
On 2014/03/24 15:17:19, Michael Starzinger wrote:
nit: The usage of the local variable "registry" is different here and
below,
this is a little bit confusing IMHO.
Done.
https://codereview.chromium.org/196103004/diff/60001/src/isolate.cc
File src/isolate.cc (right):
https://codereview.chromium.org/196103004/diff/60001/src/isolate.cc#newcode2345
src/isolate.cc:2345: JSObject::NormalizeProperties(obj,
KEEP_INOBJECT_PROPERTIES, 8);
Now forcing into dictionary mode.
https://codereview.chromium.org/196103004/
--
--
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev
---
You received this message because you are subscribed to the Google Groups "v8-dev" group.
To unsubscribe from this group and stop receiving emails from it, send an email
to [email protected].
For more options, visit https://groups.google.com/d/optout.
