Summary: | [GTK] Allow to dispatch tasks in WorkQueue with different priorities | ||||||
---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Carlos Garcia Campos <cgarcia> | ||||
Component: | Web Template Framework | Assignee: | Nobody <webkit-unassigned> | ||||
Status: | RESOLVED LATER | ||||||
Severity: | Normal | CC: | benjamin, cmarcelo, commit-queue, mcatanzaro, zan | ||||
Priority: | P2 | Keywords: | Gtk | ||||
Version: | 528+ (Nightly build) | ||||||
Hardware: | Unspecified | ||||||
OS: | Unspecified | ||||||
Attachments: |
|
Description
Carlos Garcia Campos
2015-06-09 00:46:58 PDT
Created attachment 254554 [details]
Patch
Attachment 254554 [details] did not pass style-queue:
ERROR: Source/WTF/wtf/WorkQueue.h:77: Extra space before ( in function call [whitespace/parens] [4]
ERROR: Source/WTF/wtf/WorkQueue.h:78: Extra space before ( in function call [whitespace/parens] [4]
ERROR: Source/WTF/wtf/glib/WorkQueueGLib.cpp:114: Extra space before ( in function call [whitespace/parens] [4]
ERROR: Source/WTF/wtf/glib/WorkQueueGLib.cpp:121: Extra space before ( in function call [whitespace/parens] [4]
ERROR: Source/WTF/wtf/glib/WorkQueueGLib.cpp:126: Extra space before ( in function call [whitespace/parens] [4]
ERROR: Source/WTF/wtf/glib/WorkQueueGLib.cpp:130: More than one command on the same line [whitespace/newline] [4]
ERROR: Source/WTF/wtf/glib/WorkQueueGLib.cpp:133: Extra space before ( in function call [whitespace/parens] [4]
Total errors found: 7 in 5 files
If any of these errors are false positives, please file a bug against check-webkit-style.
In the test case, why name the nonmember variables with the m_ prefix? (In reply to comment #3) > In the test case, why name the nonmember variables with the m_ prefix? Good question, there's no reason other than I copy-pasted from Tools/TestWebKitAPI/Tests/WTF/WorkQueue.cpp :-P We are no longer using this for the disk cache, so I guess we don't really need it for now. RESOLVED LATER? RESOLVED LATER. |