[webkit-reviews] review granted: [Bug 15738] a few SunSpider script improvements : [Attachment 16920] patch

[bugzilla-daemon at webkit.org]

Adam Roben <aroben at apple.com> has granted Darin Adler <darin at apple.com>'s
request for review:
Bug 15738: a few SunSpider script improvements
http://bugs.webkit.org/show_bug.cgi?id=15738

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

------- Additional Comments from Adam Roben <aroben at apple.com>
+system("$jsShellPath", "-f", $prefixFile, "-f", $resultsFile, "-f",
"resources/sunspider-analyze-results.js");

You could remove the quotes from around $jsShellPath while you're at it.

+		 $foundOutput = 0;
+		$foundStart = 1;
+		 $result = "[{\n";

Looks like you've got some tabs here.

	} else {
-	    if (/^[}][]]$/) {
+	     print;

Did you intend to insert this print statement?

+		 s/;*$//;


Do we really expect to have multiple semi-colons here?

r=me