[webkit-reviews] review granted: [Bug 80823] [GStreamer] run AudioFileReader in a nested loop : [Attachment 132230] Patch
bugzilla-daemon at webkit.org
bugzilla-daemon at webkit.org
Fri Mar 16 10:01:00 PDT 2012
Martin Robinson <mrobinson at webkit.org> has granted Philippe Normand
<pnormand at igalia.com>'s request for review:
Bug 80823: [GStreamer] run AudioFileReader in a nested loop
https://bugs.webkit.org/show_bug.cgi?id=80823
Attachment 132230: Patch
https://bugs.webkit.org/attachment.cgi?id=132230&action=review
------- Additional Comments from Martin Robinson <mrobinson at webkit.org>
View in context: https://bugs.webkit.org/attachment.cgi?id=132230&action=review
> Source/WebCore/platform/audio/gstreamer/AudioFileReaderGStreamer.cpp:332
> + GstBus* bus = gst_pipeline_get_bus(GST_PIPELINE(m_pipeline));
> + g_signal_handlers_disconnect_by_func(bus,
reinterpret_cast<gpointer>(messageCallback), this);
> + gst_object_unref(bus);
You could use a RefPtr here with adoptRef.
> Source/WebCore/platform/audio/gstreamer/AudioFileReaderGStreamer.cpp:334
> + g_signal_handlers_disconnect_by_func(m_decodebin.get(),
reinterpret_cast<gpointer>(onGStreamerDecodebinPadAddedCallback), this);
Do you need to release m_decodeBin or does it happen later automatically?
More information about the webkit-reviews
mailing list