[Webkit-unassigned] [Bug 103006] [EFL][WK2] Missing the routine to check the validation for workqueue

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Thu Nov 22 01:43:42 PST 2012


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





--- Comment #13 from Jongseok Yang <js45.yang at samsung.com>  2012-11-22 01:45:43 PST ---
(In reply to comment #12)
> (From update of attachment 175619 [details])
> View in context: https://bugs.webkit.org/attachment.cgi?id=175619&action=review
> 
> > Source/WebKit2/Platform/efl/WorkQueueEfl.cpp:82
> > +        for (size_t i = 0; i < workItemQueue.size(); ++i) {
> 
> calculating size each iteration?
> 
> > Source/WebKit2/Platform/efl/WorkQueueEfl.cpp:83
> > +            Function<void()> function = workItemQueue[i];
> 
> Sorry if I missed something, but is it mandatory to have 'function' variable here? Could it be just 'workItemQueue[i]()' after m_isValid check?

It is to meet the script "check-webkit-style". 
I got the below result from the script when removing this line.

Source/WebKit2/Platform/efl/WorkQueueEfl.cpp:83:  Line contains only semicolon. If this should be an empty statement, use { } instead.  [whitespace/semicolon] [5]

-- 
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