Automatic update of common submodule

From f363b32 to f49c55e

Back to development

Release 1.9.1

ges-launch: unref GApplication properly

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

project_: improve get_uri doc

The return value of ges_project_get_uri should be freed
after usage.
Differential Revision: https://phabricator.freedesktop.org/D1142

Automatic update of common submodule

From ac2f647 to f363b32

tests: Make sure to run python tests against the build in tree

ges: Don't remove track elements from clips when removing from layer

And reuse the same previously created element when adding the clip
back to a layer, avoiding losing all setting done on clip children
in that situation

This is a behaviour change but previous behaviour was actually totally
unexpected and people working around that weird behaviour will moste
probably not care about that change

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

title_: Do not forget to link up child_added/removed vmethod

Otherwise effect handling is broken

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

auto-transitions: Do not remove auto transitions when moving neighboor from the same group

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

ges: fix various leaks with usage of ges_timeline_element_lookup_child

Some callers forgot to unref out child, pspec or both leading to leaks.

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

video-track: don't leak restriction caps in _sync_capsfilter_with_track()

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

ges-uri-asset: GstDiscoverer can return a valid info but a non-OK result, consider this an error

The asynchronous case in ges_uri_clip_asset_request_async() already considered
it an error, do the same in ges_uri_clip_asset_request_sync().

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

nlecomposition: Fix race condition in seek handling causing deadlocks

We might receive another seek from the application while the action task is
handling a previous seek (and thus setting seeking_itself to TRUE). To prevent
this seek to go through directly instead of being added as an action, also
check if the seek event was received from our action task thread or some other
thread.

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

ges: Emit GESMetontainer::notify-meta even if value is unset

ges: Allow passing `NULL` as a value to ges_meta_container_set_meta

Fixes T7430

g-i: pass compiler env to g-ir-scanner

It's what introspection.mak does as well. Should
fix spurious build failures on gnome-continuous
(caused by g-ir-scanner getting compiler details
via python which is broken in some environments
so passing the compiler details bypasses that).

uri-clip: make uri parameter of ges_uri_clip_new () const

To avoid compiler warning when using const string to create a new
GESUriClip as string is not modified and only passed to functions which
take a const string.

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

framepositionner: add a weak ref on track element to know when it is finalized

Otherwise if frame positionner is disposed after track element has been
finalized, it will raise a critical message because we will try to
disconnect a signal handler on a freed track element object.

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

audio-source: unref private capsfilter reference on dispose

Otherwise a capsfilter reference will be leaked since it has been got
using gst_bin_get_by_name.

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

audio-source: fix indentation

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

nlecomposition: ensure elements pending to be added are not leaked

When nlecomposition is finalized with pending add action or io,
associated elements are not unreffed as they should since caller gives
us the reference when calling gst_bin_add causing them to be leaked.
So to make sure we don't leak a reference on element when adding one to
the bin, each stage (action and pending_io) hold a reference on element
and release it when stage is done.

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

nlecomposition: fix nle_composition_remove_object info message

We don't add internal bin, we remove it.

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

ges: add some g-i annotations according to documentation

Mainly (transfer xxx) and (nullable). Also fix some typo.

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

asset: fix ges_asset_set_proxy() return value documentation

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

element: Also accept GParamSpec.owner_type name as a child property prefix

Makes it simpler for python users to be able to retrieve children
properties iterating over them.

track-element: gi: skip now deprecated children property getter/setter

Those are implemented with the exact same API at the GESTimelineElement
level now, and user of those APIs with high level languages will get the
exact same API.

formatter: Prefix all children properties in the XML formatter

Otherwise it will fail on properties that are mandatorily prefixed
like the newly added deinterlacing properties

ges: Remove timeline_emit_group_removed which slipped in the API by mistake

This is formally an API break but I am sure no one ever used that and
we should make sure the method is removed as soon as possible because
it has no reason to be exposed.

container: Handle setting children properties that need prefixing

video-source: Expose deinterlace-[fields, mode, tff] child properties

Letting some control over the deinterlacing to the users

ges: fix misc g-i annotations

g-i: use only "ges/ges.h" as c-include for introspection

This is the only header which shall be included by user. Otherwise some
language using gir to generate binding, e.g Vala, will includes all
headers files in alphabetical order which causes compilation errors due
to incomplete type.

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

timeline: rename "track-element" to "track_element" in select-tracks-for-object documentation

because "track-element" is not a valid identifier for a parameter and
will cause generated binding using GIR to be invalid. For instance in
Vala.

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

video-source: Do not ever plugin avdeinterlace

It is not feature compatible with deinterlace and is not safe to use

ges: #include glib.h for G_BEGIN_DECLS

ges-multi-file-source: Fix case of standard gobject macros

ges: Add some more functions to the docs and don't scan internal headers

Someone still should look through the unused.txt for more things to add or
hide.

ges: Add G_BEGIN_DECLS around all relevant declarations in headers

title: Do not concider inpoints

It does not make sense for titles
Handle element with no inpoint handling in the timeline

Fixes https://phabricator.freedesktop.org/T7319

Automatic update of common submodule

From 6f2d209 to ac2f647

tests: Rename positionner to positioner in the tests too

ges: Fix typo by renaming positionner to positioner

It's fortunately private API

framepositionner: Initialize all fields of the meta during initialization

GstMetas are not allocated with all fields initialized to zeroes.

ges: Do not try to set read only properties

When copying and splitting clips

Fixes T7375

transition: Lower done some debug loggin level

Minor fix

titlesource: Add support for absolute positionning

tests: Add some things to .gitignore

examples/gitignore: ignore assets and play_timeline_with_one_clip binaries

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

GES: Properly split the GOptionGroup initialization

Debugging must be configuring first (before any parsing), and then
the types are initialized at the end.

Fixes issues with debugging categories not being available at the
start

editing-services: use new gst_element_class_add_static_pad_template()

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

Back to development

Release 1.8.0

Release 1.7.91

Revert "titlesource: use x/yabsolute instead of x/ypos."

This reverts commit c4356db40c6e50f7314a75ea65d46f9f21ef0a5d.

This commit was not ready and was not support to be pushed

ges: Don't emit timeline::group-removed when ungrouping outside a timeline

titlesource: use x/yabsolute instead of x/ypos.

ges-launcher: don't leak sanitized_timeline string

Summary:
sanitized_timeline is created when parsing command line,
but it isn't free'd.

Reviewers: thiblahute

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

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.