[Bug 619590] [matroskademux] Doesn't protect segment and other fields from concurrent changes from different threads

[GStreamer (bugzilla.gnome.org)]

https://bugzilla.gnome.org/show_bug.cgi?id=619590
  GStreamer | gst-plugins-good | git

Sebastian Dröge <slomo> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
 Attachment #187174|none                        |reviewed
             status|                            |

--- Comment #5 from Sebastian Dröge <slomo at circular-chaos.org> 2011-05-05 15:36:13 UTC ---
(From update of attachment 187174)
I think you don't need to take the object lock when reading the segment from
inside the streaming thread, e.g. not in gst_matroska_demux_sync_streams(),
because it's only changed from the streaming thread.

Other than that this looks good

-- 
Configure bugmail: https://bugzilla.gnome.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA contact for the bug.
You are the assignee for the bug.