[webkit-reviews] review granted: [Bug 17531] Add interactive mode to testkjs : [Attachment 21590] Proposed patch

[bugzilla-daemon at webkit.org]

Darin Adler <darin at apple.com> has granted Cameron Zwarich (cpst)
<cwzwarich at uwaterloo.ca>'s request for review:
Bug 17531: Add interactive mode to testkjs
http://bugs.webkit.org/show_bug.cgi?id=17531

Attachment 21590: Proposed patch
http://bugs.webkit.org/attachment.cgi?id=21590&action=edit

------- Additional Comments from Darin Adler <darin at apple.com>
+static const char* interactivePrompt = "> ";

This should be static const char[] or static const char* const. The declaration
as-is asks for a non-const global pointing to a const string.

+    fprintf(stderr, "	-p	   Prints formatted source code\n");

I think this option is a little silly now that it does no formatting and just
spits the code out as-is.

r=me