[Webkit-unassigned] [Bug 130027] [GLIB] Add GMainLoopSource class to wrap idle and timeout sources
bugzilla-daemon at webkit.org
bugzilla-daemon at webkit.org
Sat Mar 15 20:31:59 PDT 2014
https://bugs.webkit.org/show_bug.cgi?id=130027
Martin Robinson <mrobinson at webkit.org> changed:
What |Removed |Added
----------------------------------------------------------------------------
Attachment #226675|review? |review+
Flag| |
--- Comment #28 from Martin Robinson <mrobinson at webkit.org> 2014-03-15 20:32:16 PST ---
(From update of attachment 226675)
View in context: https://bugs.webkit.org/attachment.cgi?id=226675&action=review
Looks good for landing with Zan's suggestions.
> Source/WTF/wtf/gobject/GMainLoopSource.cpp:46
> +GMainLoopSource& GMainLoopSource::create()
> +{
> + return *new GMainLoopSource(DeleteOnDestroy);
> +}
> +
> +GMainLoopSource::GMainLoopSource()
> + : m_deleteOnDestroy(DoNotDeleteOnDestroy)
> +{
> +}
> +
> +GMainLoopSource::GMainLoopSource(DeleteOnDestroyType deleteOnDestroy)
> + : m_deleteOnDestroy(deleteOnDestroy)
> +{
> +}
This is quite clever.
> Source/WTF/wtf/gobject/GMainLoopSource.cpp:194
> + bool retval = m_socketCallback(condition);
> + if (!retval && source == m_source.get())
> + destroy();
Nit: I think you should use returnValue instead of retval.
--
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