[webkit-reviews] review denied: [Bug 22683] Improper calls to checkArgv function : [Attachment 25776] Fix of function calls

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Fri Dec 5 10:53:30 PST 2008


Eric Seidel <eric at webkit.org> has denied Zan Dobersek <zandobersek at gmail.com>'s
request for review:
Bug 22683: Improper calls to checkArgv function
https://bugs.webkit.org/show_bug.cgi?id=22683

Attachment 25776: Fix of function calls
https://bugs.webkit.org/attachment.cgi?id=25776&action=review

------- Additional Comments from Eric Seidel <eric at webkit.org>
I intentionally changed this to fix run-javascriptcore-tests --gtk, etc. to
pass --gtk along to build-jsc.

Obviously we need a better solution which handles both cases.

Which scripts are being passed an unfiltered @ARGV in the gtk port?

We could have a global flag controlling this, or we could go back to always
removing them from @ARGV (like your patch does) and add a configurationFlags()
function which knows how to produce "--debug --gtk", etc. for passing to
sub-scripts.

I'd be interested in your thoughts?


More information about the webkit-reviews mailing list