[Bug 794215] New: segment: Change back gst_segment_do_seek() to use gint64

GStreamer (GNOME Bugzilla) bugzilla at gnome.org
Fri Mar 9 19:01:21 UTC 2018


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

            Bug ID: 794215
           Summary: segment: Change back gst_segment_do_seek() to use
                    gint64
    Classification: Platform
           Product: GStreamer
           Version: git master
                OS: Linux
            Status: NEW
          Severity: enhancement
          Priority: Normal
         Component: gstreamer (core)
          Assignee: gstreamer-bugs at lists.freedesktop.org
          Reporter: nicolas at ndufresne.ca
        QA Contact: gstreamer-bugs at lists.freedesktop.org
     GNOME version: ---

During 0.11 dev at commit bdbc069348 the gst_segment_do_seek() was changed to
accept only unsigned start/stop. Though, the code will do direct comparision
with -1 and also assumes negative values for GST_SEEK_TYPE_END:

  start = segment->duration + start;

Additionally, the gst_event_new_seek() API along with the GstElement API uses
signed integer. Demuxers will pass these value unchecked to that API in order
to update their segment. Looking at the code, it will properly wrap around
resulting in the same behaviour, but it's all a bit of a lie. As this does not
affect the API, I think it's fine to switch it back to gint64.

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