[Webkit-unassigned] [Bug 101074] [GTK] Use a private struct and placement new syntax in GObject DOM bindings

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Sun Dec 9 02:17:36 PST 2012


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


Xan Lopez <xan.lopez at gmail.com> changed:

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




--- Comment #7 from Xan Lopez <xan.lopez at gmail.com>  2012-12-09 02:20:01 PST ---
(From update of attachment 177776)
This looks good. I'd just put a comment somewhere explaining why we do the Object parent class thing, and noting that this works now because the classes we care about happen to be direct subclasses of Object. That way people changing this code in the future will have a chance to get it right ;)

(Also, 'constructed' and the signal handling all seem to be garbage code from the old event listener era, we can remove that in a follow-up patch)

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