[Webkit-unassigned] [Bug 24684] Clean up WebCore based on Coverty Prevent static analysis results

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Tue Mar 24 14:22:28 PDT 2009


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


darin at apple.com changed:

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




------- Comment #19 from darin at apple.com  2009-03-24 14:22 PDT -------
(From update of attachment 28905)
I understand the desire to move asserts before the thing asserted are used, but
I don't like the idea of moving all the declarations to the top of the
function. Could you make a more limited version that doesn't move the
definitions of renderer, cachedImage, and image up to the top of the function?

In either case, r=me


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



More information about the webkit-unassigned mailing list