[Webkit-unassigned] [Bug 37836] Attempt to make new-run-webkit-tests --help more sane

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Mon Apr 19 18:35:39 PDT 2010


https://bugs.webkit.org/show_bug.cgi?id=37836





--- Comment #2 from Tony Chang (Google) <tony at chromium.org>  2010-04-19 18:35:38 PST ---
This looks great.  I am in support of this.  A couple style questions:

- Should we split all the options handling into a separate file?  It's getting
kind of long.

- I normally inline really short functions.  So maybe define _split_option_name
in _compare_option_names and define _compare_options in _add_option_group?

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.



More information about the webkit-unassigned mailing list