LGTM
http://codereview.chromium.org/8226017/diff/1/src/v8.cc File src/v8.cc (right): http://codereview.chromium.org/8226017/diff/1/src/v8.cc#newcode61 src/v8.cc:61: // TODO(rossberg): Is there a better place to put this? This is indeed not such a cool place to put it. You could consider putting it at the end of FlagList::SetFlagsFromCommandLine in flags.cc or doing something in Shell::SetOptions in d8.cc. At least that would group this code with other options/flags code. http://codereview.chromium.org/8226017/ -- v8-dev mailing list [email protected] http://groups.google.com/group/v8-dev
