[Webkit-unassigned] [Bug 42273] Convolution computation causes bad alpha channel values

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Thu Jul 15 08:55:39 PDT 2010


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





--- Comment #25 from Alex Nicolaou <anicolao at chromium.org>  2010-07-15 08:55:38 PST ---
OK I've taken another run at this.  Please take another look.

<aside>
My personal preference would be to submit simple patches that fix bugs, like the very first patch that I had for this bug; and only then make a new bug that has a patch for the pretty code whose only purpose is to clean it up. I feel like the final patch has really obscured what was once a simple fix that avoided a crash; while if we commit two patches there'll be the obvious bugfix and then the code cleanup one, and later analysis of further bugs would be easier. Given that we've gone down this road, let's just get this committed (or fix it up further as needed), but also it would be useful to me to set my expectation on separation of form and function for future changes. Thanks!
</aside>

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