[Bug 727916] bufferpool: Does not implement set_config() as documented

[GStreamer (bugzilla.gnome.org)]

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

Nicolas Dufresne <nicolas.dufresne> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|ASSIGNED                    |RESOLVED
         Resolution|                            |FIXED

--- Comment #12 from Nicolas Dufresne <nicolas.dufresne at collabora.co.uk> 2014-05-08 17:15:14 UTC ---
I've allowed myself to add a helper.


commit 23e69d98d8a6dd84faa97926cab7c2e5cf274f0a
Author: Nicolas Dufresne <nicolas.dufresne at collabora.com>
Date:   Tue Apr 15 14:17:00 2014 -0400

    pool-nego: Retry setting configuration with modified config

    Buffer pool set_config() may return FALSE if requested configuration needed
    small changes. Reget the config and try setting it again (validating the
    changes first). This ensure we have a configured pool if possible.

    https://bugzilla.gnome.org/show_bug.cgi?id=727916

commit 194db480e061f3b43c06e1f162715939e7f29c58
Author: Nicolas Dufresne <nicolas.dufresne at collabora.com>
Date:   Thu May 8 12:47:43 2014 -0400

    bufferpool: Add an helper to validate config

    When we call gst_buffer_pool_set_config() the pool may return FALSE and
    slightly change the parameters. This helper is useful to do the minial
required
    validation before accepting the modified configuration.

    https://bugzilla.gnome.org/show_bug.cgi?id=727916

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