[gstreamer-bugs] [Bug 396779] Preset interface for elements

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

  GStreamer | gstreamer (core) | Ver: HEAD CVS




------- Comment #58 from Wim Taymans  2008-03-10 16:18 UTC -------
Applied the patch, looking at it now. It seems I have the same comments as
everyone else:

* We need GST_PARAM_PRESET (or GST_PARAM_NO_PRESET) to easily filter out
unwanted params (like name and likely many other properties in encoders). A
flag to disable certain properties from being included in the presets might be
easier because then we don't need to change each element, just the ones that
have properties that should be excluded.

* A lot of code is needed for reading the file. I wonder if a combination of
GKeyFile and The GValue serialisation would simplify things.

* create_preset() indeed needs to be renamed to _randomize(). AFAIK, you give a
name to a preset with _save_preset(), which you can then get back with
_load_preset().

* do we want to define some standard tag names? Seems like a define for a
"comment" and a "description" tag would avoid having a multitude of
applications using incompatible random string for the same tags names. Maybe
state that you have to use the strings from GST_TAG?

I'll experiment a little with it.


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