[Bug 793333] matroskademux: Allow Matroska headers to be read more than once

GStreamer (GNOME Bugzilla) bugzilla at gnome.org
Fri Sep 21 19:25:24 UTC 2018


https://bugzilla.gnome.org/show_bug.cgi?id=793333

Thibault Saunier <tsaunier at gnome.org> changed:

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

--- Comment #16 from Thibault Saunier <tsaunier at gnome.org> ---
Review of attachment 373731:
 --> (https://bugzilla.gnome.org/review?bug=793333&attachment=373731)

Makes sense to me.

::: gst/matroska/matroska-demux.c
@@ +651,3 @@
           GST_ERROR_OBJECT (demux, "Invalid TrackNumber 0");
           ret = GST_FLOW_ERROR;
           break;

Why can't you keep doing the unique check here?

@@ +1514,3 @@
+      break;
+
+    case GST_MATROSKA_TRACK_TYPE_COMPLEX:

just `default:` then?

@@ +3112,3 @@
+          if (gst_matroska_read_common_tracknumber_unique (&demux->common,
+                  track->num))
+            gst_matroska_demux_add_stream (demux, track);

Please add brackets around that if too

-- 
You are receiving this mail because:
You are the QA Contact for the bug.
You are the assignee for the bug.


More information about the gstreamer-bugs mailing list