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

Use standard GST_PLUGIN_LDFLAGS for the nle plugin

Add the standard GST_PLUGIN_LDFLAGS to the configure.ac file.

Add support for static plugins builds

Release 1.5.90

Move NLE to a dedicated GstPlugin

Summary: Allowing external user to directly use it

Reviewers: Mathieu_Du

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

nle: Enhance debug logging

ges: Do not leak and uselessly create errors

And avoid parenthesis in GstObject names

ges: Fix how we handle layer vs layer-priority in the structured interface

tests: Do not use gst-structured-interface in the tests

It breaks $ make distcheck

Update mailing list in doap file

timeline: Remove transitions that can no fit into an auto transition

When activating auto transition mode

ges; Minor debug enhancement

assets: Avoid deadlock when done initialising asset

Avoid to hold the CACHE lock when setting the GTasks return values.

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

nleobject: Concider objects as 'inactive' when they have a duration == 0

track-element: Handle the case where we have only one keyframe set when interpollating keyframes

asset: Port use of deprecated GSimpleAsyncResult to GTask

Automatic update of common submodule

From f74b2df to 9aed1d7

track-element: Fix splitting bindings and add unit tests

structured-interface: Better handle CLOCK_TIME type from GstStructures

element: Implement a paste method

Allowing user to copy paste clips very easily

timeline: Disable movement that lead to 2 transition at a position

We should never let 3 objects to overlap at a same position, for that
we introduce a "rollback" feature and whenever such an editing happens,
we rollback object position to whatever it was before the move.

smart-video-mixer: Always keep a ref on the mixer pad

timeline-element: Add a method to get the TrackType it interacts with

API:
  + ges_timeline_element_get_track_types

timeline: Never create transitions between rippled objects

In case of groups, we can have track elements that do not belong
directly to the moved_trackelements but will be moved as others. Never
create transition to all object that have a start > moving group start.

video-transition: Use a SmartMixer as mixer

So that the frame position metas are parsed and taken into account

track-element: Add method to remove control binding

API:
  ges_track_element_remove_control_binding

ges: Handle absolute GstDirectControlBindings