[Webkit-unassigned] [Bug 61171] Apply the ParallelJobs support to FEConvolveMatrix

[bugzilla-daemon at webkit.org]

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


Nikolas Zimmermann <zimmermann at kde.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
  Attachment #94185|review?                     |review+
               Flag|                            |




--- Comment #2 from Nikolas Zimmermann <zimmermann at kde.org>  2011-05-20 01:27:11 PST ---
(From update of attachment 94185)
View in context: https://bugs.webkit.org/attachment.cgi?id=94185&action=review

Looks great, r=me.

> Source/WebCore/platform/graphics/filters/FEConvolveMatrix.cpp:258
> +    for (int y = yEnd + 1; y>yStart; --y) {
>          for (int x = clipRight + 1; x > 0; --x) {

Style: y > yStart, please.

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