[webkit-reviews] review denied: [Bug 101349] GStreamer reference handling fix : [Attachment 172586] Patch

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Tue Nov 6 07:45:51 PST 2012


Martin Robinson <mrobinson at webkit.org> has denied Thiago Santos
<thiago.sousa.santos at collabora.com>'s request for review:
Bug 101349: GStreamer reference handling fix
https://bugs.webkit.org/show_bug.cgi?id=101349

Attachment 172586: Patch
https://bugs.webkit.org/attachment.cgi?id=172586&action=review

------- Additional Comments from Martin Robinson <mrobinson at webkit.org>
View in context: https://bugs.webkit.org/attachment.cgi?id=172586&action=review


> Source/WebCore/platform/graphics/gstreamer/WebKitWebSourceGStreamer.cpp:220
> -    GRefPtr<GstPadTemplate> padTemplate =
adoptGRef(gst_static_pad_template_get(&srcTemplate));
> +    GRefPtr<GstPadTemplate> padTemplate =
adoptGRef(WTF::refGPtr(gst_static_pad_template_get(&srcTemplate)));

I think it would make more sense to simply avoid using adoptGRef here.


More information about the webkit-reviews mailing list