[Bug 758233] New: mpdparser: remove unused functions gst_mpdparser_get_chunk_by_index and gst_mpdparser_find_segment_by_index

[GStreamer (GNOME Bugzilla)]

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

            Bug ID: 758233
           Summary: mpdparser: remove unused functions
                    gst_mpdparser_get_chunk_by_index and
                    gst_mpdparser_find_segment_by_index
    Classification: Platform
           Product: GStreamer
           Version: git master
                OS: Linux
            Status: NEW
          Severity: normal
          Priority: Normal
         Component: gst-plugins-bad
          Assignee: gstreamer-bugs at lists.freedesktop.org
          Reporter: florin.apostol at oregan.net
        QA Contact: gstreamer-bugs at lists.freedesktop.org
     GNOME version: ---

gst_mpdparser_get_chunk_by_index and gst_mpdparser_find_segment_by_index are
unused. Should we remove them?

If we decide to keep them, there are a few problems to be fixed:
- gst_mpdparser_find_segment_by_index does not set the repeat field.
- gst_mpdparser_get_chunk_by_index, when asked to retrieve chunk indexChunk,
will retrieve the next one (the first thing it does is to increment
indexChunk). The function has no comments and is not used, so I can't figure
out if this is correct or not. From an API point of view, is completely
non-intuitive.

Once a decision to remove/fix is taken, I can implement the changes.

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