[gstreamer-bugs] [Bug 627476] New profile library and encoding plugin

[GStreamer (bugzilla.gnome.org)]

https://bugzilla.gnome.org/show_bug.cgi?id=627476
  GStreamer | gst-plugins-base | git

--- Comment #19 from Stefan Kost (gstreamer, gtkdoc dev) <ensonic at sonicpulse.de> 2010-12-07 14:25:42 UTC ---
Just for reference the branch with the latest changes (as the patches don't
apply anymore):
http://cgit.freedesktop.org/~bilboed/gst-plugins-base/tree/?h=profile

Thanks for adding the examples:
* In the "Loading a profile from disk" example it is not obvious where the
"profilename" comes from. Should apps scan a particular dir for available
profiles? Also see my comment on
gst_encoding_target_load/gst_encoding_target_save below. 

A small ambiguity is that in the example you call the
gst_encoding_target_load_from() with a parameter called uir, where in the API
docs you refer to it as a path. Would be good to fix either one.

Regarding the API:

* gst_video_encoding_profile_new()
the 'vfr' parameter is not docuemnted and I wonder if you could perhaps spell
it out. The parameter is documented in GstVideoEncodingProfile.

* if there is a gst_stream_encoding_profile_new() and
gst_video_encoding_profile_new(), shouldn't there be a
gst_audio_encoding_profile_new() too? Or is it so that right now
GstAudioEncodingProfile==GstVideoEncodingProfile (aka no specific parameters
for audio)?

So basically the only open Item for me is, how an application would look up
available targets.

* gst_encoding_target_load/gst_encoding_target_save is not implemented. I
believe specifying the locations (like done for GstPreset) would make sense.

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