[gstreamer-bugs] [Bug 353121] New: gst_query_new_segment should initialize all structure fields

[GStreamer (bugzilla.gnome.org)]

Do not reply to this via email (we are currently unable to handle email
responses and they get discarded).  You can add comments to this bug at
http://bugzilla.gnome.org/show_bug.cgi?id=353121
 GStreamer | gstreamer (core) | Ver: HEAD CVS

           Summary: gst_query_new_segment should initialize all structure
                    fields
           Product: GStreamer
           Version: HEAD CVS
          Platform: Other
        OS/Version: Linux
            Status: UNCONFIRMED
          Severity: normal
          Priority: Normal
         Component: gstreamer (core)
        AssignedTo: gstreamer-bugs at lists.sourceforge.net
        ReportedBy: kaffeetisch at gmx.de
         QAContact: gstreamer-bugs at lists.sourceforge.net
     GNOME version: Unspecified
   GNOME milestone: Unspecified


gst_query_new_segment currently only initializes the format field, but not
rate, start_value, and stop_value.  This leads to assertions if you
gst_query_parse_segment before you gst_query_set_segment.  Other query
constructors also initialize all fields even if their value isn't known yet,
for example gst_query_new_position and gst_query_new_duration.

Patch attached.


-- 
Configure bugmail: http://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.