A few more comments. LGTM if all comments are addressed.
http://codereview.chromium.org/10807024/diff/2001/src/isolate.h File src/isolate.h (right): http://codereview.chromium.org/10807024/diff/2001/src/isolate.h#newcode56 src/isolate.h:56: class CompilerProxy; Left over? I can't find any CompilerProxy class. http://codereview.chromium.org/10807024/diff/2001/src/isolate.h#newcode1294 src/isolate.h:1294: CompilerProxy* compiler_proxy_; ditto. http://codereview.chromium.org/10807024/diff/2001/src/smart-pointers.h File src/smart-pointers.h (right): http://codereview.chromium.org/10807024/diff/2001/src/smart-pointers.h#newcode139 src/smart-pointers.h:139: #endif // V8_SMART_ARRAY_POINTERS_H_ You probably mean V8_SMART_POINTERS_H_ in this comment. http://codereview.chromium.org/10807024/ -- v8-dev mailing list [email protected] http://groups.google.com/group/v8-dev
