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

[bugzilla-daemon at webkit.org]

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





--- Comment #19 from Jongseok Yang <js45.yang at samsung.com>  2012-11-22 04:09:22 PST ---
(In reply to comment #18)
> (From update of attachment 175643 [details])
> View in context: https://bugs.webkit.org/attachment.cgi?id=175643&action=review
> 
> >> Source/WebKit2/Platform/efl/WorkQueueEfl.cpp:83
> >> +            {
> > 
> > This { should be at the end of the previous line  [whitespace/braces] [4]
> 
> I wish we had 'bool WorkQueue::isValid() const' encapsulating all of this

I think that it is better to use just "function" variable just to avoid the fail for check-webkit-style.
WorkQueue::isValid() is redundant if it is required just to avoid the fail for check-webkit-style.

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