[gstreamer-bugs] [Bug 498004] preset is ignored by lame plugin

[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=498004

  GStreamer | gst-plugins-ugly | Ver: 0.10.6




------- Comment #4 from Eddy Pronk  2007-12-02 11:06 UTC -------
Created an attachment (id=100039)
 --> (http://bugzilla.gnome.org/attachment.cgi?id=100039&action=view)
patch to change the way properties are set.

This patch is probably incomplete, but it demonstrates a different way to
modify properties. In the current implementation all parameters are set in
gst_lame_setup() whether or not they are being set by gst_lame_set_property().
I just wanted to set lame_set_preset() not touching all other settings. I moved
all calls to lame_set_* to gst_lame_set_property(). If I understood correctly,
GstLame hold a shadow copy of all properties, which is still useful for reading
properties. The way it deals with defaults needs work. Let me know what you
think.  I'd be happy to make it more complete.


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