[gstreamer-bugs] [Bug 401072] Move libgimme-codec helper functions to GStreamer

[GStreamer (bugzilla.gnome.org)]

Do not reply to this via email (we are currently unable to handle email
responses and they get discarded).  You can add comments to this bug at
http://bugzilla.gnome.org/show_bug.cgi?id=401072

  GStreamer | gst-plugins-base | Ver: HEAD CVS


Tim-Philipp Müller changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
  Attachment #81366|0                           |1
        is obsolete|                            |




------- Comment #2 from Tim-Philipp Müller  2007-01-29 14:07 UTC -------
Created an attachment (id=81433)
 --> (http://bugzilla.gnome.org/attachment.cgi?id=81433&action=view)
updated patch

New patch, addressing some of the issues raised on IRC:

The main issue is the XID argument to gst_plugin_install_{sync|async}. We
should replace this with something that's not platform-specific. or get rid of
it completely. IMHO the latter is not a good idea, because it's something
distros seem to want (to make it transient to the application window while the
plugin installation is in progress) and they'll do it one way or another, so
let's just make it part of the API instead of forcing them to patch stuff all
over the place to make this work.

So instead of an xid we now pass a GstPluginInstallContext argument, and apps
can set their XID on the context before passing it in. This allows us to add
variants that make sense on win32/OSX/BeOS/whatever later. Also, it allows us
to easily add other useful things later on.

Also improved the docs a bit:
 - mention that gst_install_plugins_sync() should usually not be used
 - some additinos to gst_install_plugins_async()
 - for both mention where the installer detail strings come from

Also added a function:
 - gst_install_plugins_installation_in_progress()


-- 
Configure bugmail: http://bugzilla.gnome.org/userprefs.cgi?tab=email