After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 779106 - flvdemux: remove duplicated segment initialization
flvdemux: remove duplicated segment initialization
Status: RESOLVED FIXED
Product: GStreamer
Classification: Platform
Component: gst-plugins-good
1.10.x
Other Mac OS
: Normal minor
: 1.11.91
Assigned To: GStreamer Maintainers
GStreamer Maintainers
Depends on:
Blocks:
 
 
Reported: 2017-02-23 00:42 UTC by Changbok
Modified: 2017-04-20 11:45 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
flvdemux: remove duplicated segment initialization (952 bytes, patch)
2017-02-23 00:45 UTC, Changbok
committed Details | Review

Description Changbok 2017-02-23 00:42:47 UTC
When gst_flv_demux_init function is invoked, segment initialization iscalled redundantly by gst_flv_demux_cleanup. So it needs to remove duplicated segment initialization in gst_flv_demux_init.
Comment 1 Changbok 2017-02-23 00:45:17 UTC
Created attachment 346522 [details] [review]
flvdemux: remove duplicated segment initialization
Comment 2 Tim-Philipp Müller 2017-04-20 11:45:21 UTC
commit 3d19d6d8bc5a08f56c070d49b8255130add258f0
Author: ChangBok Chae <changbok.chea@gmail.com>
Date:   Thu Feb 23 09:31:36 2017 +0900

    flvdemux: remove duplicated segment initialization
    
    It's also done in gst_flv_demux_cleanup().
    
    https://bugzilla.gnome.org/show_bug.cgi?id=779106