pad: don't fallback to caps queries with proxy pads

A proxy-pad should always proxy the caps related queries
and events to its down or upstream peers on the other side
of the element. Falling back to a caps query seems wrong.

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

harness: minor doc fixes

segment: Replaced gst_segment_to_position with gst_segment_position_from_running_time

gst_segment_to_position might cause confusion, especially with the addition of
gst_segment_position_from_stream_time . Deprecated gst_segment_to_position
now, and replaced it with gst_segment_position_from_running_time.

Also added unit tests.

segment: Added unit tests for gst_segment_position_from_stream_time

segment: gst_segment_to_stream_time: Renamed 'result' to 'stream_time'

Renamed the "result" variable to "stream_time" for better readability.

segment: Added gst_segment_position_from_stream_time()

gst_segment_position_from_stream_time() will convert stream time into a
position in the segment so that gst_segment_to_stream_time() with that
position returns the same stream time. It will return -1 if the stream time
given is not inside the segment.

segment: Rewording of struct field descriptions

The new wording makes it easier to understand exactly what each field of the
GstSegment struct represents.

event: Make sure that timestamp + diff in QoS events is never smaller than 0

When a running-time-offset is stored in the event, it could become smaller
than 0 although the event is otherwise correct. This can happen when pad
offsets are used.

To prevent this, we set the timestamp to -diff, so that in the end the sum of
both is exactly 0.

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

uri: Add test for correct absolute URI handling in gst_uri_from_string_with_base()

If the second parameter is an absolute URI, the base should have no effect and
the second parameter should be returned again.

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

Back to development

Release 1.6.0

Update .po files

ptpclock: Fix error leak during failures

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

funnel: Fix racy state change

Iterator may need to be resynced, for instance if pads are released
during state change.

got_eos should be protected by the object lock of the element, not of
the pad, as is the case throughout the rest of the funnel code.

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

bin: element: Ignore activate result for removed pads on state change

This fixes a race where a state change may return failure if it has
request pads that are deactivated and removed (and thus have no
parent) at the same time as the element changes state and (de)activates
its pads.

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

harness: don't crash when adding a sink-harness without h->sinkpad

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

basetransform: Print buffer PTS when submitting an input buffer

inputselector: Fix buffer leak in sync_streams & cache_buffers mode

After doing gst_pad_push() in case of sync_streams and cache_buffers,
if the buffer can not be kept in cache, it should be unreffed to avoid
memory leackage.

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

gst: Documentation typo fix in caps & pad APIs

gst_caps_can_intersect() & GST_PAD_IS_ACCEPT_INTERSECT()
documentation typo fix.

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

Release 1.5.91

Update .po files

po: Update translations

downloadbuffer, benchmarks: fix error leaks in failure code paths

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

check: Fix indentation

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

bufferpool: sprinkle some allow-none and out annotations for g-i

bin: fix typo in log message when threadpool alloc fails

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

configure: Check for socketpair() in -lsocket too

On e.g. QNX it is in an external library, not libc.

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

Revert "docs: Make sure gst_debug_bin_to_dot_data() is documented"

This reverts commit 0dffeb03018d12be522c2d97aaaf8102153bd7c0.

The commit is erroneous and documents the function twice.

docs: Make sure gst_debug_bin_to_dot_data() is documented

Thanks to Nirbheek Chauhan <nirbheek@centricular.com> for pointing this
out.

harness: Fix race for gst_harness_element_ref

In order for gst_harness_new_full to be MT-safe the increase and
decrease of HARNESS_REF must be MT-safe. This allows for creating
multiple harnesses from different threads wrapping the same element.

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

harness: Allow-none for custom stress init func

It should be allowed to not have a function to initialize the user data
since it's often not necessary; it may already be initialized.

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

docs: remove signal that no longer exists from docs

po: update for translated string changes

tools: gst-launch: fix --exclude command line option

This has not worked (as in: crashed) since 2005, so
perhaps it should just be removed instead.

Revert "queue2: Process SEEKING query"

This caused problems with oggdemux when queue2 was
operating in queue mode and the souphttpsrc upstream
is not seekable because the server doesn't support
range requests. It would then still claim seekability
and then things go wrong from there.

This reverts commit 7b0b93dafe4ac547552cdb66ade5d8aa0405e7b4.

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

harness: misc bugfixes

1. Get a list of pad templates from the element class, not the
factory. This allows us to interact with test-elements that does
not have a factory.

2. Use the pad_template_caps in caps-queries when caps is not set
explicitly on the pad. Not doing so is simply wrong, and prohibits
interactions with special templates used for testing.

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

tests: event: fix build failure

gst/gstevent.c:250:5: error: ‘for’ loop initial declarations are only
allowed in C99 or C11 mode

bin: Make sure we don't add/remove a bin to/from itself

Doing so would deadlock from trying to acquire the object lock twice

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

basetransform: Reconfigure before propose_allocation

There exist cases where a reconfigure event was propagated from
downstream, but caps didn't change. In this case, we would
reconfigure only when the next buffer arrives. The problem is that
due to the allocation query being cached, the return query parameters
endup outdated.

In this patch we refactor the reconfigurating code into a function, and
along with reconfiguring when a new buffer comes in, we also reconfigure
when a query allocation arrives.

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

basesrc-test: Fix race testing segment update

As this test is using a short sleep (GST_USECOND, which is 10ms
in microsecond), sometimes that EOS event is received before the
loop in basesrc have run _do_seek() and pushed the update segment.
To solve this issue, we wait for the initial segment (and flush it)
then we wait for the second segment before sending EOS.

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

bin: Add NLE to GST_PLUGIN_PATH

Release 1.5.90

Update .po files

po: Update translations

baseparse: avoid tag list spam if upstream provides bitrate tags already

Explicitly keep track again whether upstream tags or parser tags
already contain bitrate information, and only force a tag update
for a bitrate if we are actually going to add the bitrate to the
taglist later. This fixes constant re-sending of the same taglist,
because upstream provided a bitrate already and we didn't add it,
so we didn't save the 'posted' bitrate, which would then in turn
again trigger the 'bitrate has changed too much, update tags'
code path. Fixes tag spam with m4a files for example.

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

debugutils: bring the dot style a bit closer to what we use in the docs

Use round corners for bins and elements. Put sink pads on the left and src pads
on the right of elements.

baseparse: fix tag handling

In 0.10 there were no sticky events, and all tag events
sent would just be merged with the previously-received
tags. In 1.x we have sticky events, and the tags in the
tag event(s) should at all times carry the complete tags,
so we can't just push some tags and then just push tags
with just bitrates to update the bitrates, etc.

Instead we need to keep track of the upstream stream tags
received, of the tags set by the video decoder subclass,
and send an updated tag event with the combined tags
including our own bitrate tags (if applicable) whenever
the upstream tags, the subclass tags or any of our bitrates
change.

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

baseparse: add API for subclass to set tags

This is needed so that we can do proper tag handling
all around, and combine the upstream tags with the
tags set by the subclass and any extra tags the
base class may want to add.

API: gst_base_parse_merge_tags()

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

baseparse: save upstream stream tags

We'll need those later.

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

baseparse: minor code simplification

Use gst_pad_peer_query_duration() and remove a few
unnecessary levels of indentation. Rest of code might
looks a bit questionable, but leave it as is for now.

pad: Break sticky event array iterations if the type is bigger than the one we look for

Microoptimization we can do because the array is sorted by type.

gstpad: Add a new GST_PROBE_HANDLED return value for probes

In some cases, probes might want to handle the buffer/event/query
themselves and stop the data from travelling further downstream.

While this was somewhat possible with buffer/events and using
GST_PROBE_DROP, it was not applicable to queries, and would result
in the query failing.

With this new GST_PROBE_HANDLED value, the buffer/event/query will
be considered as successfully handled, will not be pushed further
and the appropriate return value (TRUE or GST_FLOW_OK) will be returned

This also allows probes to return a non-default GstFlowReturn when dealing
with buffer push. This can be done by setting the
GST_PAD_PROBE_INFO_FLOW_RETURN() field accordingly

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

gstversion: Add missing include in .in file.

Headers: add missing includes.

docs: add the new pad accept-template flag to the docs

docs: section entry missing for gst_direct_control_binding_new_absolute

tests: pad: tests for accept-caps default handling

Check if all the default 4 accept-caps possibilities are working:
subset or intersect check and query-caps or template caps comparisons.

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

pad: add GST_PAD_FLAG_ACCEPT_TEMPLATE

It will make the default accept-caps handler use the pad template
caps instead of the query-caps result to check if the caps is
acceptable. This is aligned with what the design docs says the
accept-caps should do (be non-recursive) and should be faster. It
is *not* enabled by default, though.

API: GST_PAD_FLAG_ACCEPT_TEMPLATE
API: GST_PAD_IS_ACCEPT_TEMPLATE
API: GST_PAD_SET_ACCEPT_TEMPLATE
API: GST_PAD_UNSET_ACCEPT_TEMPLATE

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

check: Rename states unit test

Makes it easier to differentiate from other modules states unit test

basetransform: rework accept-caps

According to the design docs:
The ACCEPT_CAPS query is not required to work recursively, it can simply
return TRUE if a subsequent CAPS event with those caps would return
success.

So make it a shallow check instead of recursivelly check downstream.

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

basetransform: remove some dead code

Doesn't seem like it is going to get back to life anytime soon

Also removes a {} block that was likely used to keep the dead
code around.

basetransform: respect accept-caps intersect flag

GstPad has a flag for suggesting if the accept-caps
query should use intersect instead of the default
subset caps operation to verify if the caps would be
acceptable.

basetransform currently always uses the subset check and
this patch makes it honor the flag for using intersect
if it is set.

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

basetransform: remove unreachable return statement

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

tests: ignore new harness test binary

datetime: accept just a time as ISO 8601 string and use today's date then

If no date and only a time is given in gst_date_time_new_from_iso8601_string(),
assume that it is "today" and try to parse the time-only string. "Today" is
assumed to be in the timezone provided by the user (if any), otherwise Z -
just like the behavior of the existing code.

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

harness: add _set_forwarding function

To be able to disable the slightly "magic" forwarding of the
necessary events between the harnesses.

Also introduce a new test-suite for GstHarness, that documents the
feature, and should hopefully expand into documenting most of the
features the harness possesses.

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

devicemonitor: fix provider leak

pad: Fix previous commit

We want to get the caps query *result*

deviceprovider: Add method to hide devices from a provider

Add methods to add/remove the providers that should be hidden by this
provider. Also make a method to get a list of hidden providers.

This makes it possible to have multiple systems monitor the same devices
and remove duplicates.

Add a property to see all devices, even duplicate ones from hidden
providers.

pad: get_allowed_caps() should go through both pads

The previous implementation was doing a direct call to the peer pad,
which resulted in query probes never being called on the original pad.

Instead of that, get the peer pad caps by using gst_pad_peer_query()
which will call probes in the expected fashion.

value: free caps during failure

While calling gst_value_deserialize_sample, if there is a failure
after caps is ref'ed, then caps is getting leaked. Hence checking for
caps in fail: goto condition and unref'ing it

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

registry: Add plugins to the registry we are loading and not default one

When running gst_registry_scan_plugin_file we were losing the
information about the registry being loaded and ended up adding the
plugin to the default registry which was not correct.

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

basesink: Only drop buffer if their PTS is out of segment

As of now, even for stream completly inside segment, there is no
guarantied that the DTS will be inside the segment. Specifically
for H.264 with B-Frames, the first few frames often have DTS that
are before the segment.

Instead of using the sync timestamp to clip out of segment buffer,
take the duration from the start/stop provided by the sub-class, and
check if the pts and pts_end is out of segment.

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

harness: don't run code inside g_assert

Even though asserts can't be disabled in GstHarness, Coverity still
complains about running code inside them. Moving the code to outside the
g_asserts().

CID #1311326, #1311327, #1311328

devicemonitor: get a list of currently monitored providers

Get a list of the currently monitored providers.

pad: fix invalid unref after IDLE probe on non-OK flow return

In case there is an IDLE probe fired from gst_pad_push_data and it
doesn't return GST_FLOW_OK, the code jumps to the probe_stopped
label which tries to unref the data object. However, at this point
the data object belongs downstream and must not be touched.
By setting data = NULL, the code skips this unref.

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

buffer: Fix the name of the parentbuffermeta debug category.

Don't use 'glbufferrefmeta' as the debug category for the
parent buffer meta.

queue2: not update upstream size with negative value

upstream_size can be negative but queue->upstream_size is unsigned type.
to get a chance to update queue->upstream_size in gst_queue2_get_range()
it should keep the default value.

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

buffer: Remove extra debug symbol from exports

Don't export the debug variable for the parent_buffer_meta.

This was accidentally exported and shouldn't be public

filesink: use GST_INFO_OBJECT for more detail

Helps to distiguish multiple filesinks.

docs: info: remove 0.8 terminology from log level description

We don't "iterate" bins or pipelines any more.

tests: baseparse: fix buffer leak in unit test

Fixes make check-valgrind

doc/seekflags: Fix cross references

This fixes miss-use of @ instead of % to refer to enumeration
values.

docs: add a few more new symbols and defines

doc/capsfilter: Document filtering modes

This is documentation for the HTML documentation.

doc/filesink: Add BufferMode enumeration

This is purely for documentation purpose. This way the values will
show up in the HTML documentation.

doc/gsthardness: Fix typo in GstAllocationParams

It's not GstAllocatorParams but GstAllocationParams.

doc/gstharness: Remove unknown parameter

sink_elenment_name is not a parameter of gst_harness_add_sink_harness()
function, but still it show up in documentation.

capsfilter: Only remember previous filter caps if they were actually used for something

If nobody ever saw the previous filter caps, nothing could've negotiated with
them and we can just pretend they never existed at all.

capsfilter: When switching caps change modes, forget all previous caps

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.