[gstreamer-bugs] [Bug 582564] [controller] Use ordered GSequence instead of GList in the interpolation control source

[GStreamer (bugzilla.gnome.org)]

If you have any questions why you received this email, please see the text at
the end of this email. Replies to this email are NOT read, please see the text
at the end of this email. You can add comments to this bug at:
  http://bugzilla.gnome.org/show_bug.cgi?id=582564

  GStreamer | gstreamer (core) | Ver: git

Jonathan Matthew changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |notverysmart at gmail.com




------- Comment #5 from Jonathan Matthew  2009-05-17 14:33 UTC -------
This seems to be causing a lot of critical warnings in rhythmbox.  

For the first track played, per buffer:
  GLib-CRITICAL **: g_sequence_get_begin_iter: assertion `seq != NULL' failed

and for every track after that, when fading in:
  GLib-CRITICAL **: g_sequence_free: assertion `seq != NULL' failed

Partial stack trace for the first warning:

#7  0xb724ff3a in IA__g_sequence_get_begin_iter (seq=0x0) at
/build/buildd-glib2.0_2.20.1-2-i386-hGzT8z/glib2.0-2.20.1/glib/gsequence.c:1067
#8  0xb7cdde1b in _interpolate_linear_get_double (self=0xa7bbc28, timestamp=0,
ret=0xafffae00) at gstinterpolation.c:606
#9  0xb7cde666 in interpolate_linear_get_double (self=0xa7bbc28, timestamp=0,
value=0xafffaeb8) at gstinterpolation.c:606
#10 0xb7cea059 in gst_control_source_get_value (self=0xa7bbc28, timestamp=0,
value=0xafffaeb8) at gstcontrolsource.c:96
#11 0xb7cdc928 in gst_controller_sync_values (self=0xab507a8, timestamp=0) at
gstcontroller.c:697
#12 0xb7ce9a85 in gst_object_sync_values (object=0xab8c308, timestamp=0) at
gsthelper.c:184
#13 0xb4c6f756 in volume_before_transform (base=0xab8c308, buffer=0xabd28a0) at
gstvolume.c:730

At this point there would not be any control points set.

The second occurs in gst_interpolation_control_source_unset_all.  There's no
check to see if self->priv->values is NULL in _unset_all (or in _unset).


-- 
See http://bugzilla.gnome.org/page.cgi?id=email.html for more info about why you received
this email, why you can't respond via email, how to stop receiving
emails (or reduce the number you receive), and how to contact someone
if you are having problems with the system.

You can add comments to this bug at http://bugzilla.gnome.org/show_bug.cgi?id=582564.