Summary: | Split QueueStatus server into modules | ||||||
---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Adam Barth <abarth> | ||||
Component: | Tools / Tests | Assignee: | Adam Barth <abarth> | ||||
Status: | RESOLVED FIXED | ||||||
Severity: | Normal | CC: | eric, evan, levin, webkit.review.bot | ||||
Priority: | P2 | ||||||
Version: | 528+ (Nightly build) | ||||||
Hardware: | All | ||||||
OS: | All | ||||||
Attachments: |
|
Description
Adam Barth
2009-12-18 23:04:44 PST
Created attachment 45225 [details]
Patch
style-queue ran check-webkit-style on attachment 45225 [details] without any errors.
Comment on attachment 45225 [details]
Patch
I dont' really know the patterns to comment. This looks better than what we had.
This would be easier to review if you made each file a derivative of the original file. Hard to convince git to do this. One way is a) cp origin-file new-file1; cp origin-file new-file2; etc. git add, commit -a, svn dcommit. b) now do your edits and let us review the patch. This LGTM though. Attachment 45225 [details] was posted by a committer and has review+, assigning to Adam Barth for commit.
Looks like this was already landed as bug 52381. Silly Adam. ;) |