[Webkit-unassigned] [Bug 20932] Notable leaks in QtWebkit - from Qt library

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Mon Dec 8 12:44:21 PST 2008


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


eric at webkit.org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
  Attachment #25012|review?                     |review-
               Flag|                            |




------- Comment #17 from eric at webkit.org  2008-12-08 12:44 PDT -------
(From update of attachment 25012)
I'm confused.  You move the location of * (to match WebKit style I assume?) yet
you don't clean up all the other style violations in this file (and add some of
your own with this patch -- like { } around single line ifs).  I'm not sure
what style the QtLauncher code is supposed to be in, but I assume WebKit style.
 I didn't actually look at the substance of the patch.


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