[Bug 623799] [tagsetter] Configurable metadata output

[GStreamer (bugzilla.gnome.org)]

https://bugzilla.gnome.org/show_bug.cgi?id=623799
  GStreamer | gstreamer (core) | git

--- Comment #29 from Stefan Kost (gstreamer, gtkdoc dev) <ensonic at sonicpulse.de> 2011-03-31 21:05:51 UTC ---
(In reply to comment #28)
> There's now a GstTagXmpWriter interface which does exactly what it's supposed
> to. I don't really see any major use cases remaining, nor do I have better
> ideas how to approach this with generic API, so I think we should just close
> this bug or rename it back to 'configurable xmp output', unless someone wants
> to pursue the generic approach further.

I just now saw that plan b was done in bug #645167. Whats is the issue with a
more generic API? Elements that implement tagsetter would need two extra
methods:
GList *gst_tag_setter_get_variants(GstTagSetter *ts)
 if return list is NULL -> no variants
gst_tag_setter_set_variants(GstTagSetter *ts, GList *variants)
 set the variants to use, all unknown variants are ignored. one should really
use 
 _get_variants(), filter and call _set_variants()

Each variant would be a group in the sense of a pick one or pick any. The group
would be a list of string and description.

Basically what thiago did, but hiding the structure from the api if that is
desired.

I really thing the XmpConfig is not something we want to have in a release.

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