basetransform: Return FLOW_FLUSHING if negotiation fails during shutdown

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

harness: Fix indendation

basetransform: Avoid increasing query reference

gst_query_find_allocation_meta() requires the query to be
writable to work. This patch ensure avoids taking a reference
on the query, so we can now check if a certain allocation meta
is present.

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

docs: fix description of gst_buffer_extract_dup()

No GBytes involved.

concat: dot not reset pad states too early

Resetting the flushing state of the pads at the end of the
PAUSED_TO_READY transition will make pads handle serialized
queries again which will wait for non-active pads and might
cause deadlocks when stopping the pipeline.

Move the reset to the READY_TO_PAUSED instead.

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

harness: add functions for adding sub-harnesses directly

By introducing gst_harness_add_src_harness and gst_harness_add_sink_harness
we collect all sub-harness setup in one function, making the previous
sub-harness creation functions now calls these directly, and making it
much easier (and less error-prone) to add your own src or sink-harness
using the more generic harness-creation functions.

baseparse: Don't override gst_segment_do_seek()

This line has no purpose, clearly gst_segment_do_seek() is doing
the right job, also, having the start time (a timestamp) be that
same as time (the stream time) is quite odd.

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

baseparse: Fix extrapolation of seeksegment.stop

The stop shall be relative to start if extrapolated from the
duration.

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

devicemonitor: do start and stop outside of the lock

Release the monitor lock when calling the provider start/stop methods.
Because we release the lock now, We need to make sure we check the
cookie again and keep track of started and removed providers.

deviceprovider: small cleanups

Protect against wrong arguments.
Clean up the header file indentation.

devicemonitor: keep order of providers and devices

The deviceproviders are added to the array sorted by their rank. Make
sure we keep this ordering when removing a provider.

We use _prepend to collect the devices, use g_list_reverse to get the
devices in the right order; sorted by rank and in the same order as
returned by the provider.

harness: fix indentation

harness: fix pad template leak

docs: drop reference to sourceforge mailing list adress

harness: don't re-establish the harness sink and src pads

Given that the element has the possibility to have one, they should
already be there.

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

harness: Improve detection of element type

The element flag does not indicate wether a bin should be tested as a
source or as a sink, eg. a bin with the sink flag may still have a
source pad and a bin with the source flag may have a sink pad. In this
case it is better to determine the element type by looking at the
available pads and pad templates.

Also rename srcpad and sinkpad where it actually represents
element_srcpad_name and element_sinkpad_name.

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

harness: Forward sticky events to sink harness

Fixes issue where if a sink harness was added late the sticky events
would not be forwared.

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

harness: make header nicer to read

docs: add new function to API docs

device: add generic struct with properties

Add a generic structure to hold any additional properties about the
device.

tee: fix typo in allow-not-linked property description

docs: bus: mention main loop requirement in gst_bus_add_watch() docs

task: add function guard for _set_lock() and fix guard for _join()

Should only access the object structure after checking
it's valid in gst_task_join().

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

protection: implement meta transform function

Copy the GstMeta contents over to the new buffer.

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

baseparse: estimate duration on EOS

For files which are smaller than 1.5 seconds, the duration
estimation does not happen. So the duration will always be
displayed as 0. Updating the duration on EOS when the estimation
has not happened already

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

adapter: change log message properly

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

docs: add StreamidDemux to documentation

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

adapter: fix to get valid (buffer_)list

get_list/get_buffer_list should be done with buffers in adapter remaining
while take_list/take_buffer_list flushes each buffer one by one.

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

adapter: unit test for new get_(buffer_)list

baseparse: put buffer in a correct state after gst_adapter_get_buffer call

We must make the buffer writable to write its PTS and DTS, and also
reset its duration.

The behaviour is now the same as before commit c3bcbadd, except metas
might still be attached to the buffer extracted from the adapter.

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

harness: fix indentation and replace stress test function macros

These screw with indentation and seem a bit trivial. Just copy'n'paste.

meta: transform_func: return FALSE if not supported or failed

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

identity: refactor and add tests using GstHarness

Writing a test for unscheduling the gst_clock_id_wait inside the
identity element, found an invalid read, caused by removing the clock-id
when calling _unschedule instead of letting the code calling _wait remove
the clock-id after being unscheduled.

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

harness: make sure g_assert() statements are always active

We have code with side effects inside g_assert()s, so make
sure those are always enabled here (they might otherwise
get disabled for release builds).

harness: rename GstHarnessPrepareBuffer -> GstHarnessPrepareBufferFunc

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

docs: add GstHarness to documentation

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

check: Add GstHarness convenience API for unit tests

http://gstconf.ubicast.tv/videos/gstharness-again-a-follow-up/

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

baseparse: reverse playback in pull mode

right now reverse playback is disabled in pull mode.
enabling the code for the same and changing a bit of logic
to make reverse playback work.

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

baseparse: add reverse playback test in pull mode

add test for reverse playback in pull mode and compare
the buffers being received in sink chain to make sure
the playback is allright

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

Revert "scripts: Allow passing make flags to git-update.sh"

This reverts commit ab5fdd72129ea61e8dff51cdc0afcccac03ebc2b.

We can use the MAKEFLAGS environment variable to pass options to make,
so avoid adding another mechanism that could be confusing.

pad: Clarify pad probe return type documentation

basesink: Shouldn't drop buffer when sync=false

Shouldn't drop buffer when sync=false

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

scripts: Allow passing make flags to git-update.sh

Mostly adding this for add a -jN as appropriate while building.

queue: avoid slice allocs/frees for each item

Microoptimisation: Let GstQueueArray store our
item struct. That way we don't have to alloc/free
temporary QueueItem slices for every item we want
to put into the queue.

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

queuearray: allow storing of structs in addition to pointers

This way we don't have to allocate/free temporary structs
for storing things in the queue array.

API: gst_queue_array_new_for_struct()
API: gst_queue_array_push_tail_struct()
API: gst_queue_array_peek_head_struct()
API: gst_queue_array_pop_head_struct()
API: gst_queue_array_drop_struct()

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

Automatic update of common submodule

From f74b2df to 9aed1d7

pad: Enforce NEED_PARENT flag also for chain

The check for the presence of the parent in the presence of
the NEED_PARENT flag was missing for the chain function. Also keep
a ref on the parent in case the pad is removed mid-chain.

docs: update for two missing elements

Concat was not linked and streamiddemux was missing.

docs: another sweep canonicalizing the plugin docs sections file

Use underscores for capsfilter macros. Correct the type-name for fakesrc
if we ever implement the enum.

docs: order and canonicalize the -sections.txt file

Have all sections in alphabetical order. Also make the macro order consistent.
This is a preparation for generating the file. Remove GET_CLASS macro for
typefine element, since it is not used and the header is not installed.

memory: Add missing field initializers to GstMapInfo

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

inputselector: remove always-true check

event can't be NULL, it has been dereferenced by GST_EVENT_TYPE (), and no
case frees the pointer. Remove unnecessary check which will always be True.

CID #1308955

transform: Also copy POOL metas and make sure to copy over metas when creating subbuffers

POOL meta just means that this specific instance of the meta is related to a
pool, a copy should be made when reasonable and the flag should just not be
set in the copy.

adapter: Also copy POOL metas and make sure to copy over metas when creating subbuffers

POOL meta just means that this specific instance of the meta is related to a
pool, a copy should be made when reasonable and the flag should just not be
set in the copy.

buffer: Don't copy "memory" metas unconditionally

Don't copy memory metas if we only copied part of the buffer, didn't
copy memories or merged memories. In all these cases the memory
structure has changed and the memory meta becomes meaningless.

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

Revert "buffer: Don't copy POOLED and memory metadata unconditionally"

This reverts commit 7a08fa5ec4804f104e9aa9f458322f6eb49a7e49.

buffer: Don't copy POOLED and memory metadata unconditionally

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

baseparse: Use new gst_adapter_get_buffer() API instead of gst_adapter_map()

This preserves GstMeta properly unless the subclass does special things. It's
enough to make h264parse's stream-format/alignment conversion pass through
metas as needed.

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

adapter: Add get variants of the buffer based take functions

Main difference to gst_adapter_map() for all practical purposes is that
GstMeta of the buffers will be preserved.

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

adapter: Copy over GstMeta from the input buffers to the output

All functions that return a GstBuffer or a list of them will now copy
all GstMeta from the input buffers except for meta with GST_META_FLAG_POOLED
flag or "memory" tag.

This is similar to the existing behaviour that the caller can't assume
anything about the buffer flags, timestamps or other metadata. And it's
also the same that gst_adapter_take_buffer_fast() did before, and what
gst_adapter_take_buffer() did if part of the first buffer or the complete
first buffer was requested.

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

ptp: Init function can take a NULL interfaces array

tests: caps: fix test_intersect_flagset failure

test_intersect_flagset fails because when caps is being
created, flags and mask are being cast to uint64 while
they should be uint. This results in invalid memory access
or a segfault.

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

scripts: Fix GST_VALIDATE_PLUGIN_PATH

It moved recently

basetransform: Fix up documentation of transform_meta vfunc

By default we copy all metas that have no tags.

directcontrolbinding: fix ABI break

Structure size was increased without adjustment of the padding.

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

task: guard against NULL task function

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

funnel: add "forward-sticky-events" property

It is useful to avoid sending sticky event on stream changes.

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

Back to development

Release 1.5.2

Update .po files

tests: gstutils: fix wrong description of test element

It is a fakesink with request pads, not a source

bufferpool: Fixed compiler warning

The pool variable was unused when buidling with debug disabled.

po: Update translations

win32: Update .def file for new API

basesink: need to deep-copy last buffer list in drain

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

bufferlist: Warn if copying a buffer fails in gst_buffer_list_copy_deep()

bufferlist: add new api gst_buffer_list_copy_deep

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

gstcheck: Print newline in message handler

The message handler is supposed to print a newline after the message
just like the default message handler.

inputselector: Handle different duration track selection

Support track switch from EOS track to non-EOS one.

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

pad: Clear EOS flag after received STREAM_START event

Clear EOS flag after received STREAM_START event

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

tests: gstutils: add tests for gst_element_get_compatible_pad

Adds tests for gst_element_get_compatible_pad for when it has to
request pads.

Note that these tests don't cover the case when it has to request
a pad that already exists.

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

utils: use caps when getting a compatible pad by template

Do not ignore the caps argument when requesting a pad by template.

This is particularly harmful when the pad caps query by default
returns ANY so it will match the first template instead of the
one that actually intersects with the caps.

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

gstsample.h: Include gstbufferlist.h now that it uses GstBufferList

concat: when releasing pad, send EOS appropriately.

Previously, concat sent an EOS if there was a next pad.

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

concat: Add adjust-base property

This disables the segment.base adjustments, which is useful if downstream
takes care of base adjustments already (example: a combination of concat
and streamsynchronizer)

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

basesink: Unset the last buffer list if we only got a buffer

Also remember any preroll buffer list.

sample: Add new API to the docs

basesink: enable to get last sample including buffer list if needed

In case of a buffer list rendering, last-sample is not updated.
It needs to be updated and enable to get buffer list from last-sample.

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

sample: add gst_sample_set/get_buffer_list apis

Allowed to set/get buffer list to sample if needed

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

test.py: remove accidentally committed file

doc: Unify Since mark for attribute and enum

As this show up as prose in the doc, simply make it consistent
and "arguable" nicer to read.

doc: Remove parenthesis around Since:

This confuse the parser, hence it does not endup in the doc and the
index properly.

Add GstParentBufferMeta

A core meta which helps implement the old concept
of sub-buffering in some situations, by making it
possible for a buffer to keep a ref on a different
parent buffer. The parent buffer is unreffed when
the Meta is freed.

This meta is used to ensure that a buffer whose
memory is being shared to a child buffer isn't freed
and returned to a buffer pool until the memory
is.

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

gi: Use INTROSPECTION_INIT for --add-init-section

This new define was added to common. The new init section fixed
compilation warning found in the init line that was spread across
all files.

Automatic update of common submodule

From 6015d26 to f74b2df

clock: Fix _STIME_FORMAT macros

This macro didn't work well as it relied on the sign on the last
divided number (number of days). This value is most of the time
zero, and zero is considered positive in printf. Instead, deal with
the sign manually, and resuse the original macros for the rest. This
actually simplify the macro a lot.

typefindelement: reset segment only once streaming has stopped

Fixes the occasional criticals in the discoverer unit test.

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

doc: Add more missing symbols in lib-sections.txt

These where causing broken links.

doc: Fix reference to unknown type GstNetAddress

doc: Include and fix GstControlPoint