Created attachment 213852 [details] [review]
Patch to implement the RST parsing

This should be ported to the new mpegts library (which exports all section
information and descriptors on the bus).

See gst-libs/gst/mpegts/ and tests/examples/mpegts/

Are you planning to update the patch, Michał?

ping ?

Created attachment 278419 [details] [review]
mpegts: dvb: add rst table parsing

Adds RST (running status table) parsing

Created attachment 278420 [details] [review]
tsparser: add rst table dumping

One question for those patches, should we prepend DVB to the structures and enums? Or as our DVB isn't using that we just keep with the same naming scheme?

Review of attachment 278419 [details] [review]:

::: gst-libs/gst/mpegts/gst-dvb-section.c
@@ +1241,3 @@
+
+  /* Skip 3 bytes of initial offset */
+  data += section->section_length;

That looks wrong

::: gst-libs/gst/mpegts/gst-dvb-section.h
@@ +358,3 @@
+  guint16 service_id;
+  guint16 event_id;
+  guint8  running_status;

Reuse the GstMpegtsRunningStatus enum for that field

Created attachment 282353 [details] [review]
mpegts: dvb: add rst table parsing

Adds RST (running status table) parsing

Created attachment 282354 [details] [review]
tsparser: add rst table dumping

ping

-- GitLab Migration Automatic Message --

This bug has been migrated to freedesktop.org's GitLab instance and has been closed from further activity.

You can subscribe and participate further through the new bug through this link to our GitLab instance: https://gitlab.freedesktop.org/gstreamer/gst-plugins-bad/issues/65.