Release 1.7.90

nle: Set the NleOperation flags to NLE_OBJECT_OPERATION

Reviewed By: thiblahute

Differential Revision: https://phabricator.freedesktop.org/D770

ges: Register scaletempo::rate as a rate changing property

Handle changing playback rate

Before this patch, NLE and GES did not support NleOperations (respectively
GESEffects) that changed the speed/tempo/rate at which the source plays. For
example, the 'pitch' element can make audio play faster or slower. In GES 1.5.90
and before, an NleOperation containing the pitch element to change the rate (or
tempo) would cause a pipeline state change to PAUSED after that stack; that has
been fixed in 1.5.91 (see #755012 [0]). But even then, in 1.5.91 and later,
NleComposition would send segment events to its NleSources assuming that one
source second is equal to one pipeline second. The resulting early EOS event
(in the case of a source rate higher than 1.0) would cause it to switch stacks
too early, causing confusion in the timeline and spectacularly messed up
output.

This patch fixes that by searching for rate-changing elements in
GESTrackElements such as GESEffects. If such rate-changing elements are found,
their final effect on the playing rate is stored in the corresponding NleObject
as the 'media duration factor', named like this because the 'media duration',
or source duration, of an NleObject can be computed by multiplying the duration
with the media duration factor of that object and its parents (this is called
the 'recursive media duration factor'). For example, a 4-second NleSource with
an NleOperation with a media duration factor of 2.0 will have an 8-second media
duration, which means that for playing 4 seconds in the pipeline, the seek
event sent to it must span 8 seconds of media. (So, the 'duration' of an
NleObject or GES object always refers to its duration in the timeline, not the
media duration.)

To summarize:

* Rate-changing elements are registered in the GESEffectClass (pitch::tempo and
  pitch::rate are registered by default);
* GESTimelineElement is responsible for detecting rate-changing elements and
  computing the media_duration_factor;
* GESTrackElement is responsible for storing the media_duration_factor in
  NleObject;
* NleComposition is responsible for the recursive_media_duration_factor;
* The latter property finally fixes media time computations in NleObject.

NLE and GES tests are included.

[0] https://bugzilla.gnome.org/show_bug.cgi?id=755012

Differential Revision: https://phabricator.freedesktop.org/D276

Automatic update of common submodule

From b64f03f to 6f2d209

group-added and group-removed signals added

Differential Revision: https://phabricator.freedesktop.org/D619

Back to development

Release 1.7.2

uninstalled.pc: add support for non libtool build systems

Currently the .la path is provided which requires to use libtool as
mentioned in the GStreamer manual section-helloworld-compilerun.html.
It is fine as long as the application is built using libtool.

So currently it is not possible to compile a GStreamer application
within gst-uninstalled with CMake or other build system different
than autotools.

This patch allows to do the following in gst-uninstalled env:
gcc test.c -o test $(pkg-config --cflags --libs gstreamer-1.0 \
  gst-editing-services-1.0)
Previously it required to prepend libtool --mode=link

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

Fix and test priority of TrackElement after splitting

And make sure we properly handle transitions in that case

ges: Give better names to nleobjects

tests: extend the AM_TESTS_ENVIRONMENT from check.mak

To get the CK_DEFAULT_TIMEOUT defined for all tests

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

Automatic update of common submodule

From 86e4663 to b64f03f

titlesource: Add properties for text dimensions.

trackelement: Make use of read-only children properties.

Read only properties will throw a GLib warning like this
when accessed with "set_child_property":

Warning: g_object_set_property: property 'text-x' of object class 'GstTextOverlay' is not writable

nle: Turn composition structural issue into ERROR on the bus

Those error are really critical and we are then enable to keep
working. Just post an ERROR message on the bus and let the
application deal with it.

Reviewed-by: Mathieu Duponchelle <mathieu.duponchelle@opencreed.com>
Differential Revision: https://phabricator.freedesktop.org/D740

track-element: Rely on nleobject to be created at construct time

Avoiding all the pending_xx dance and making the code simpler.

This is now possible thanks to the various recent refactoring.

Thanks to that the user is able to set_child_property on objects
that are not in GESTrack yet, as expected.

Reviewed-by: Thibault Saunier <thibault.saunier@collabora.com>
Differential Revision: https://phabricator.freedesktop.org/D739

effect: Determine the effect type as soon as possible

Making it possible to create the nleobject right at the creation
of the element.

Reviewed-by: Thibault Saunier <thibault.saunier@collabora.com>
Differential Revision: https://phabricator.freedesktop.org/D738

title-clip: Return default GESTitleSource value if no child set yet

In get_property we should return the default values if
we have not created any GESTitleSource yet
(instead of segfaulting).

And fix GESTitleSource default values!

Reviewed-by: Thibault Saunier <thibault.saunier@collabora.com>
Differential Revision: https://phabricator.freedesktop.org/D737

ges: track-element: Try to create NleObject as soon as possible

This way we have informations about the content of the
children as soon as possible.

Most code paths where already ready to handle that as we use it for
copying clips.

Fix framepositionner to properly handle that (it would have broke
with copied clips before).

Reviewed-by: Thibault Saunier <thibault.saunier@collabora.com>
Differential Revision: https://phabricator.freedesktop.org/D736

timeline: Avoid possible crash disposing the timeline

g-i: fix init section to avoid compiler warnings

container: Update start if adding a child that as a start < current start

Reviewed-by: Thibault Saunier <thibault.saunier@collabora.com>
Differential Revision: https://phabricator.freedesktop.org/D629

timeline: Fix infinite loop on dispose

Reviewed-by: Thibault Saunier <thibault.saunier@collabora.com>
Differential Revision: https://phabricator.freedesktop.org/D628

group: Make deep copying actually copy deep

Allowing pasting groups paste exactly what had been copied

And not the new version of the contained objects

This technically breaks the C API but this is a new API and I believe
and hope nobody is using it right now.

Reviewed-by: Thibault Saunier <thibault.saunier@collabora.com>
Differential Revision: https://phabricator.freedesktop.org/D616

tests_: Add a simple python copy/paste test for groups

Integrating python tests in the build system

And cleanup configure.ac

Reviewed-by: Thibault Saunier <thibault.saunier@collabora.com>
Differential Revision: https://phabricator.freedesktop.org/D601

Do not install ges-smart-video-mixer.h

it should always have been private

Reviewed-by: Thibault Saunier <thibault.saunier@collabora.com>
Differential Revision: https://phabricator.freedesktop.org/D617

Revert "timeline-element: Do not consider not serializable elements when getting top element"

This commit was causing issue where we were reporting the toplevel
element as an element but that element was actually in another
not serialized group. That is very tricky to handle for end users
as they are not guaranteed the toplevel clips were actually not
contained in another element.

This reverts commit ceb82ba3028332987d8d5251f98b4896120aa59b.

Reviewed-by: Thibault Saunier <thibault.saunier@collabora.com>
Differential Revision: https://phabricator.freedesktop.org/D627

nlecomposition: use correct type for flush_seqnum.

nleghostpad: use GST_SEGMENT_FORMAT

This isn't 2005 anymore.

ges-asset: Don't dereference NULL proxy assets when resolving fails

CID 1346531

ges: Fix various g-i warnings

ges-track-element: Rename control-binding-reomved signal to control-binding-removed

Strictly speaking an API change but nobody on the Internet seemed to have used
the signal with the typo in the name.

Back to development

Release 1.7.1

ges-validate: Fix compiler warning caused by usage of wrong enum type

ges-validate.c:237:22: error: implicit conversion from enumeration type
      'GESEdge' to different enumeration type 'GESEditMode'
      [-Werror,-Wenum-conversion]
  GESEditMode edge = GES_EDGE_NONE;
              ~~~~   ^~~~~~~~~~~~~
ges-validate.c:277:41: error: implicit conversion from enumeration type
      'GESEditMode' to different enumeration type 'GESEdge'
      [-Werror,-Wenum-conversion]
              new_layer_priority, mode, edge, position))) {
                                        ^~~~

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

timeline-element: Do not consider not serializable elements when getting top element

Those are temporary elements that should not be considered when dealing
with the hierarchy of objects.

Fixes T3455

uri-clip: Copy sources child properties when resetting asset

examples: Move all examples to the root dir and create foldersdir per language

+ Add some markdown files to link between languages
+ Add a simple 'play timeline with one clip" example in C and python

configure: Use -Bsymbolic-functions if available

While this is more useful for libraries, some of our plugins with multiple
files and some internal API can also benefit from this.

Revert "project: Call asset_added in the first signal emition stage"

This reverts commit 08f927ca68f71530a32846b6da19eac9dc439a2c.

That commit was breaking the API and could break other people's code.

asset: Add a way to set asset as "needing reload"

Allowing application to force the asset system to recheck if an
asset has been "fixed" and can be used again

API:
    + ges_asset_needs_reload

Differential Revision: https://phabricator.freedesktop.org/D584

project: Call asset_added in the first signal emition stage

Differential Revision: https://phabricator.freedesktop.org/D520

Implement asset proxying support

API:
  ges_asset_set_proxy
  ges_asset_get_proxy
  ges_asset_list_proxies
  ges_asset_get_proxy_target

Differential Revision: https://phabricator.freedesktop.org/D504

Automatic update of common submodule

From b319909 to 86e4663

bash-completion: Disable during "make distcheck" as this requires installing files outside the prefix

automake requires all files to be installed inside the prefix. bash-completion
requires the files to be in a specific directory given by a pkg-config file.
As such those two are having incompatible requirements and we just disable
bash-completion installation for the time being when running "make distcheck".

Nonetheless things like "make install" with e.g. a DESTDIR or a private
installation into a user's directory will fail as in both cases the
bash-completion data would be tried to be installed system-wide.

Revert "build: fix make distcheck."

This reverts commit 462727d6d825b6e67119e6b8ea47d9e18cc22bdf.

This "fix" broke the build on Windows, where both prefix and datadir are
absolute paths and as such we would concatenate two absolute paths and fail.

tests: Properly setup GST_PLUGIN_PATH in test environement

Dist gst-editing-services.doap

asset: Add a method to retrieve the GError of an asset loaded with error

API:
    ges_asset_get_error

project: Add a 'asset-loading' signal

ges: Set restriction caps in the audio source caps filter

Otherwise we could have not negotiated errors in audiomixer when
the channel/channel-mask do not match

Differential Revision: https://phabricator.freedesktop.org/D493
Reviewed-by: Mathieu Duponchelle <mathieu.duponchelle@opencreed.com>

formatter: Do not serialize top effect priorities

We just need to make sure they are always serialized in the right
order (which is the case) and de serializing them will lead to the
right behaviour.

We should not serialize the priority as the priority of the source
itself depends on the action having been done on the parent clip,
and we do not serialize the source priorities (and should not, GES
should just do the right thing).

Differential Revision: https://phabricator.freedesktop.org/D491

launcher: Make sure to not activate validate twice when simply loading a scenario

uri-clip: Make sure to instantiate an asset to back GESUriClip-s

Automatic update of common submodule

From b99800a to b319909

Use new GST_ENABLE_EXTRA_CHECKS #define

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

Automatic update of common submodule

From 9aed1d7 to b99800a

Update GLib dependency to 2.40.0

nlecomposition: free closure actions when disposing

Summary:
After invoking GClosure, the item of action list becomes
orphan so it lost a chance to be freed. In addition, even
when disposing, the list of actions has few items so we
have to check the list.

Reviewers: thiblahute

Projects: #gstreamer_editing_services

Reviewed By: thiblahute

Differential Revision: https://phabricator.freedesktop.org/D324

nlecomposition: fix wrong argument order of GClosureNotify

Summary:
_free_action should follow GClosureNotify type.

```
void
(*GClosureNotify) (gpointer data,
                   GClosure *closure);
```

Reviewers: thiblahute

Projects: #gstreamer_editing_services

Reviewed By: thiblahute

Differential Revision: https://phabricator.freedesktop.org/D323

track: mixing_operation is handled by its parent

Summary:
Normally, mixing_operation is created and added to nlecomposition
as a child element so it will be freed when nlecomposition is removed
from a track.

Reviewers: thiblahute

Projects: #gstreamer_editing_services

Differential Revision: https://phabricator.freedesktop.org/D319

nleoperation: don't leak iterator

Summary: Once an iterator is created, it should be freed after usage.

Reviewers: thiblahute

Projects: #gstreamer_editing_services

Reviewed By: thiblahute

Differential Revision: https://phabricator.freedesktop.org/D318

asset: simplify if-statement in cache_set_loaded

Summary:
Manual iteration can be replaced with foreach function.
In addition, this patch fixes mismatched GFunc type for
g_list_foreach and adds debug cateory for gst-asset for
convenient debugging.

Reviewers: thiblahute

Reviewed By: thiblahute

Differential Revision: https://phabricator.freedesktop.org/D312

uri-asset: do not reuse a passed GError pointer

Summary: A passed GError is re-allocated when discoverer has no information.

Reviewers: thiblahute

Projects: #gstreamer_editing_services

Reviewed By: thiblahute

Differential Revision: https://phabricator.freedesktop.org/D302

xml-formatter: handle dispose properly

Summary:
To dispose properly, a child object should call same function
of parent class.

Reviewers: thiblahute

Differential Revision: https://phabricator.freedesktop.org/D311

base-xml-formatter: properly handle GFile from wrong uri

Summary:
g_file_new_for_uri never fails so GFile always has valid pointer.
And fix a bug of double unref from D303.

Reviewers: thiblahute

Differential Revision: https://phabricator.freedesktop.org/D310

don't leaks caps and converted strings

Summary:
Valgrind reports trivial leakages related to handling
objects and their converted strings.

Reviewers: thiblahute

Differential Revision: https://phabricator.freedesktop.org/D303

track: add gaps when going from READY to PAUSED.

Summary:
The backend commits itself automatically in these cases, so track
needs to do so too.

Reviewers: thiblahute

Reviewed By: thiblahute

Differential Revision: https://phabricator.freedesktop.org/D94

editing-services: Fix memory leaks when context parse fails

When g_option_context_parse fails, context and error variables are not getting free'd
which results in memory leaks. Free'ing the same.

And replacing g_error_free with g_clear_error, which checks if the error being passed
is not NULL and sets the variable to NULL on free'ing.

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

nleoperation: don't leak srcpad

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

project: fix a pointer for error message

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

project: don't leak GFileInfo

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

timeline-element,track,framepositionner: don't leak internal object

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

structured-interface: introduce TRY_GET_STRING

TRY_GET uses gst_structure_get. However, if boxed or
string pointer is retrieved by gst_structure_get,
it should be freed properly.

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

uri-asset: don't leak uri string

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

Revert "title-source: Force format with alpha channels out of videotestsrc"

This reverts commit 7d1e1010728a5348674bb9053de6b095cb824984.

This commit was never meant to be committed (at least *not* on master).

structure-parser: define GES_STRUCTURE_PARSER macro

And fix trivial leakages of internal list structure.

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

title-source: Force format with alpha channels out of videotestsrc

Making sure the user can set a background of the title with an alpha
channel.

Working around https://bugzilla.gnome.org/show_bug.cgi?id=755482 for
the 1.6 branch.

Back to development

Release 1.6.0

ges: Avoid emitting 'child-added/removed' when signal emission stops addition

In the GESTimeline, TrackElement addition to a clip might get cancelled
(and thus the element gets removed), we need to make sure users do not
get wrong signals.

Also document the fact that user should connect to container::child-added
with g_signal_connect_after.

nle{composition,object}: remove unused allocation & trivial leakages

nlecomposition allocates unused 'UpdateCompositionData' and it
causes leakages.

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

pipeline: don't leak GstPad

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

pitivi-formatter: don't leak internal hash table

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

video-transition: Make compositor background transparent

Allowing further mixing downstream

video-transition: Add a framepositioner at the end of the transitio

So downstream compositor knows the zorder of the various streams

Release 1.5.91

ges: Namespace NLE utils function into GES

Avoiding name clashes when built statically

nle: Avoid unsetting srcpad target after the srcpad is already freed

That leaded to segfaults

nle: Stop wrongly set operation segment base time

Inside the composition we actually do not need to have any notion
of what the timing outside the compositon as we already tweak the segment
base time outside the composition. This code was only there to work
around https://bugzilla.gnome.org/show_bug.cgi?id=753196

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

nlecomposition: don't leak internal hashtable

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

nleobject: don't leak srcpad when disposing

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

timeline: don't leak pad in private structure

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

ges-launcher: don't leak GError

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

ges-launcher: fix double free when argument is invalid

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

video-source: Use the priority being set to compute zorder

video:transition: Set mixer pad zorder

video-source: Make sure to set framepositionner zorder when creating it

And fix a computation bug where we would be having mixing order
reversed between layers.

And make sure that the positionner does not mix up Transition handling
of the zorder

video-source: Simply set framepositionner->zorder = self->priority

Summary:
Making the code simpler and handling the transition case
where elements are in the same layer (which was failing
/setting same zorders until now).

Reviewers: Mathieu_Du

Differential Revision: https://phabricator.freedesktop.org/D237