LGTM when it lints.
http://codereview.chromium.org/1456001/diff/4001/5001 File src/liveedit.cc (right): http://codereview.chromium.org/1456001/diff/4001/5001#newcode450 src/liveedit.cc:450: static bool IsStubCode(Code* code) { I will suggest making a IsJSFunctionCode which checks == with Code::FUNCTION instead. The term "stub" is a bit overloaded. http://codereview.chromium.org/1456001/diff/4001/5001#newcode465 src/liveedit.cc:465: if (!IsStubCode(shared_info->code())) { Maybe (shared->code()->kind() == Code::FUNCTION) is just as good here? http://codereview.chromium.org/1456001/diff/4001/5001#newcode472 src/liveedit.cc:472: Handle<Code> new_original_code = Factory::CopyCode(compile_info_wrapper.GetFunctionCode()); Long line, please make sure it lints. http://codereview.chromium.org/1456001 -- v8-dev mailing list [email protected] http://groups.google.com/group/v8-dev To unsubscribe, reply using "remove me" as the subject.
