[gst-devel] [gst-cvs] gstreamer: registry: allow plugins to cache extra data in registry. Fixes #570233

[Sebastian Dröge]

Am Montag, den 08.06.2009, 11:02 +0100 schrieb Tim-Philipp Müller:
> On Sun, 2009-06-07 at 13:50 -0700, Stefan Kost wrote:
> 
> Hi,
> 
> > URL:    http://cgit.freedesktop.org/gstreamer/gstreamer/commit/?id=ed88db818b64982adcdcc29e88ad476358a0757d
> > Author: Stefan Kost <ensonic at users.sf.net>
> > Date:   Sun Jun  7 23:46:54 2009 +0300
> > 
> > registry: allow plugins to cache extra data in registry. Fixes #570233
> >
> > Add a GstStructure to GstPlugin. Plugins can retieve it in plugin_init and
> > access the cached info or build the cache and store it there.
> 
> nice that you added this functionality, but:
> 
>  - may I ask if anyone reviewed this new API before you
>    committed it? Putting a patch into bugzilla on a Saturday
>    night and committing it on Sunday doesn't really cut it
>    for non-trivial API. [*]

I've reviewed the API and talked about it with Stefan on IRC yesterday.
Also the API is quite trivial IMHO, I mean it's just setting/getting a
GstStructure to a GstPlugin and writing/reading it to the registry ;)
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 197 bytes
Desc: Dies ist ein digital signierter Nachrichtenteil
URL: <http://lists.freedesktop.org/archives/gstreamer-devel/attachments/20090608/f016c6e1/attachment.pgp>