Summary: | Remove some warnings seen when building Qt | ||||||||
---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Julien Chaffraix <jchaffraix> | ||||||
Component: | Tools / Tests | Assignee: | Nobody <webkit-unassigned> | ||||||
Status: | RESOLVED FIXED | ||||||||
Severity: | Minor | CC: | commit-queue, webkit.review.bot | ||||||
Priority: | P2 | ||||||||
Version: | 528+ (Nightly build) | ||||||||
Hardware: | All | ||||||||
OS: | All | ||||||||
Attachments: |
|
Description
Julien Chaffraix
2010-02-16 19:51:07 PST
Created attachment 48855 [details]
Proposed fix - add some checks and refactor the command testing logic
Attachment 48855 [details] did not pass style-queue:
Failed to run "WebKitTools/Scripts/check-webkit-style" exit_code: 1
WebKitTools/Scripts/webkitdirs.pm:620: Line contains tab character. [whitespace/tab] [5]
Total errors found: 1
If any of these errors are false positives, please file a bug against check-webkit-style.
Created attachment 48858 [details]
Proposed fix - add some checks and refactor the command testing logic
A tab sneaked into this change, same patch without it.
Comment on attachment 48858 [details]
Proposed fix - add some checks and refactor the command testing logic
LGTM.
Comment on attachment 48858 [details] Proposed fix - add some checks and refactor the command testing logic Clearing flags on attachment: 48858 Committed r54899: <http://trac.webkit.org/changeset/54899> All reviewed patches have been landed. Closing bug. |