[Bug 701287] gnonlin: API should be revisited before1.X first versions

GStreamer (bugzilla.gnome.org) bugzilla at gnome.org
Sun Jun 23 14:06:14 PDT 2013


https://bugzilla.gnome.org/show_bug.cgi?id=701287
  GStreamer | gnonlin | git

Thibault Saunier <thibault.saunier> changed:

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

--- Comment #14 from Thibault Saunier <thibault.saunier at collabora.com> 2013-06-23 21:06:09 UTC ---
commit 5c026dfe236c6d6493e77c3eaafa643bb0dfd7ff
Author: Thibault Saunier <thibault.saunier at collabora.com>
Date:   Thu Jun 13 20:33:00 2013 -0400

    gnlcomposition: Remove support for gaps

    The way we were handling gap was weird and not natural at all for
    users, remove it for now.

    In the long term, we should have a proper way of filling gaps in the
    API and for now only emit an STREAM_ERROR message on the bus when a
    gap is detected.

    The user is now responsible for filling gaps in the composition

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

commit 9545f1b58e42cf8a80744e7e269fbb650208f729
Author: Thibault Saunier <thibault.saunier at collabora.com>
Date:   Thu Jun 13 19:39:55 2013 -0400

    gnlcomposition: Remove uneeded indents

    Going from

    if (something) {
      do_something();
    } else {
    }

    to

    if (!something)
      return;

    do_something ();

commit bb8749f0c710a71f0afbd2499b0842d61cf0cc4d
Author: Thibault Saunier <thibault.saunier at collabora.com>
Date:   Thu Jun 13 19:33:46 2013 -0400

    gnlcomposition: Make the pipeline update protected from any child changes

    When calling the gnl_object_commit method, we need to make sure that no
    object can be changed and, it should be true until the pipeline has
    fully been udpated.

    We now need the update_pipeline function to be called with the
    COMP_OBJECTS_LOCK taken, and we do not unlock that lock during the whole
    pipeline updating process.

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

commit 43fe6ac3c0f4e9ccd6c0ff8ffc5c7cffdcf3076e
Author: Thibault Saunier <thibault.saunier at collabora.com>
Date:   Thu Jun 13 18:51:15 2013 -0400

    gnl: Setting priority == MAXUINT32 does not mean "default object"

    This was how expandables worked prior to the expandable property.

    Using MAXUINT32 has some limitation and which are fully covered by the
    expandable property. Drop this support, as we will plan on implementing
    an automatic way of filling gaps.

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

commit c99cc0ef17d5040105958bfc6085fe298cba2096
Author: Thibault Saunier <thibault.saunier at collabora.com>
Date:   Fri Jun 7 18:58:24 2013 -0400

    gnl: First implementation of the commit based API

    Currently, the only safe way to modify a composition, or an object, is to
pause
    the pipeline first. We think that this limitation eliminates many use cases
for
    GNonLin. Few video editors let you create a draft of your video by applying
    multi-cam technic. Also people could want to eventually consider doing live
    video production, which obviously require being able to change the
composition
    in playing state (and increase compositon duration as we go).

    This commit is a first step making GNL fully commit base. Objects,
Operations and
    Compositon keep a copy of modified values, and only apply those changes
    when a commit is executed. This potentially allows making the changes
    (or group of changes) fully 'atomic' without having to pause the pipeline.

    Added API:
    ----------
      GnlObject::commit action signal

    Removed API:
    ------------
      GnlComposition:update property

    + Fix the tests

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

commit d66b29d55d7506e0c0fed84439876f7e56c60127
Author: Thibault Saunier <thibault.saunier at collabora.com>
Date:   Fri Jun 7 18:17:25 2013 -0400

    gnl: Remove trailling whitespaces and tabs in .c files

commit f08303aa50283d1da784daf27521e640898217e0
Author: Thibault Saunier <thibault.saunier at collabora.com>
Date:   Fri Jun 7 11:39:17 2013 -0400

    gnl: Remove the notion of media-duration

    Rational described at https://bugzilla.gnome.org/show_bug.cgi?id=701287

commit b57873c96e23d7b33750088ad990f80a98f5658e
Author: Thibault Saunier <thibault.saunier at collabora.com>
Date:   Thu May 30 17:48:47 2013 -0400

    gnl: Rename media-start as inpoint

    Rational described at https://bugzilla.gnome.org/show_bug.cgi?id=701287

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


More information about the gstreamer-bugs mailing list