[Bug 709180] New: mpegts: [PATCH] [FIXME] Return GstMpegTsDescriptor in mpegts_get_descriptor_from_*

[GStreamer (bugzilla.gnome.org)]

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

           Summary: mpegts: [PATCH] [FIXME] Return GstMpegTsDescriptor in
                    mpegts_get_descriptor_from_*
    Classification: Platform
           Product: GStreamer
           Version: git
        OS/Version: Linux
            Status: UNCONFIRMED
          Severity: normal
          Priority: Normal
         Component: gst-plugins-bad
        AssignedTo: gstreamer-bugs at lists.freedesktop.org
        ReportedBy: jesper.larsen at ixonos.com
         QAContact: gstreamer-bugs at lists.freedesktop.org
                CC: bilboed at bilboed.com
     GNOME version: ---


Created an attachment (id=256178)
 View: https://bugzilla.gnome.org/attachment.cgi?id=256178
 Review: https://bugzilla.gnome.org/review?bug=709180&attachment=256178

Patch v1

Patch on master to correct a FIXME.

The functions mpegts_get_descriptor_from_program and
mpegts_get_descriptor_from_stream should return the GstMpegTsDescriptor instead
of a pointer to descriptor data.

I have touched the parts that uses the functions, but I don't know if the
preferred way is to pass desc->data to the macros, or if the macros should do
the ->data?

-- 
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.