LGTM Thanks for fixing this!
On 2011/08/31 19:04:00, tfarina wrote: http://codereview.chromium.org/7785027/diff/1/src/extensions/externalize-string-extension.cc
File src/extensions/externalize-string-extension.cc (right):
http://codereview.chromium.org/7785027/diff/1/src/extensions/externalize-string-extension.cc#newcode136
src/extensions/externalize-string-extension.cc:136: static ExternalizeStringExtension* externalize_extension; On 2011/08/31 14:04:48, Mikhail Naganov (Chromium) wrote: > For clarity, please initialize with NULL.
Done.
http://codereview.chromium.org/7785027/diff/1/src/extensions/externalize-string-extension.cc#newcode137
src/extensions/externalize-string-extension.cc:137: if
(!externalize_extension)
On 2011/08/31 14:04:48, Mikhail Naganov (Chromium) wrote: > externalize_extension == NULL
Done.
http://codereview.chromium.org/7785027/ -- v8-dev mailing list [email protected] http://groups.google.com/group/v8-dev
