goom: remove compiler trick

After commit 2cb6cfed22166b262ae50cb58f3ff11dd8ba91f9 there is no need to
trick the compiler anymore about the usage of variable cpuFlavour.

Automatic update of common submodule

From b99800a to b319909

audiofx: remove unused variable

Remove unsued variable have_coeffs in audiofxbaseiirfilter

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

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

flvdemux: relax creation time parsing

Parse wrong timestamps like we used to write as well,
e.g. 10:9:42, and the hour might be without a leading
zero in any case.

flvdemux: fix indentation

flvdemux: extract both creation date and time

Before we only extracted the date part.

flvmux: fix writing of creation time

Don't write time as e.g. 11:9:42

rtpj2kpay: update fragment offset

It was always being set to 0, making the resulting stream broken
for the receiver

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

qtmux: Don't unconditionally use strnlen()

It's not available on older OSX and we can as well use memchr() here.

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

auparse: Fix event memory leak

Free the event after being handled to prevent memory leak.

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

qtmux: unify raw audio caps into a single caps structure

qtdemux: add support for FFV1 coded streams in mov

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

souphttpsrc: EOS immediately if we have an empty seek segment

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

souphttpsrc: Make non-inclusive segment boundaries inclusive

The problem is that the filesrc and souphttpsrc are behaving
differently regarding the calculation of the segment boundaries. The
filesrc is using a non-inclusive boundaries, while the souphttpsrc
uses inclusive. Currently the hlsdemux calculates the boundaries as
inclusive, so for this reason there is no problem with the souphttpsrc,
but there is an issue in the filesrc.

The GstSegment is non-inclusive, so the proposed solution is to use
non-inclusive boundaries in the hlsdemux in order to be consistent.
Make the change in the hlsdemux, will break the souphttpsrc, which
will expect inclusive boundaries, but the hlsdemux will offer
non-inclusive. This change makes sure that the non-inclusive
boundaries are converted to inclusive.

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

souphttpclientsink: Add the retry and retry-delay properties

These allow a failed request to be retried after the given number of seconds
instead of failing the pipeline. Take account of the Retry-After header if
present. Add retries parameter that controls the number of times an HTTP
request will be retried before failing.

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

qtdemux: fix caps leak

If the QtDemuxStream are re-used they may already have caps which used
to be leaked.

Reproduced using the
validate.dash.playback.seek_forward.dash_exMPD_BIP_TC1 validate
scenario.

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

qtdemux: Fix taglist memory leak

Free the stream and its sub items instead of just the stream

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

qtmux: Allow negotiating to S8 as a raw format but stop making it best choice

Negotiation to audio/x-raw,format=S8 was not possible because S8 does
not have a bit order so we ended up doing `if (!entry.fourcc) goto refuse_caps;`

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

qtmux: Add prores support

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

tests: add GST_PLUGINS_BASE_LIBS for flvdemux check

So it pulls in the right libgsttag-1.0.

goom/goom2k1: remove obsolete left over files

They now use the new GstAudioVisualizer base class
from gst-plugins-base/gst-libs/gst/pbutils

Also fixed undefined reference to gst_audio_visualizer_get_type
Added GST_PLUGINS_BASE_LIBS to Makefile.am and re-order LIBADD.

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

mpegaudioparse: Fix buffer memory leak during failures

mapped buffer is not being unmapped during failures

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

souphttpclientsink: Check if soup message is created

If soup message is not created then the same should not be passed
on, which is resulting in segfault. Hence throwing a warning message
and returning

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

souphttpclientsink: Check if location being set is valid

Adding a check in set_property to find if the location uri is valid
and printing warning if not valid.

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

souphttpclientsink: Fix memory leaks during failures

freeing streamheader_buffers and sent_buffers during failure cases.

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

souphttpclientsink: Replace redundant free_buffer_list function

Removing free_buffer_list and replacing it with already available function
g_list_free_full

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

check: Don't forget base CFLAGS for flvdemux check

elements/flvdemux.c:25:25: fatal error: gst/tag/tag.h: No such file or directory

matroskamux: Create a TIME segment when creating streamable output

Related to https://bugzilla.gnome.org/show_bug.cgi?id=754435 which
does the same for flvmux.

flvdemux: output speex vorbiscomment as a GstTagList

This is what speexdec expects.

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

flvmux: GST_BUFFER_OFFSETs should be GST_BUFFER_OFFSET_NONE

Or else flvdemux don't understand it

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

flvmux: use time segment and copy timestamps when streamable

Add a basic test using speex data to verify timestamping.

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

flvdemux: speex is also always 16KHz

This is just a cosmetic change for the logs, since the right caps
for Speex is being set elsewhere.

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

rtpmanager: Add 'source-stats' to stats and notify

Add statitics from each rtp source to the rtp session property.
'source-stats' is a GValueArray where each element is a GstStructure of
stats for one rtp source.

The availability of new stats is signaled via g_object_notify.

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

rtpsession: Implement sending of reduced size RTCP packets

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

audiofx: Remove unused variable

Remove unused variable 'degree' in audiodynamic

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

qtdemux: Fix memory leak for corrupted file

Free brands before overriding them.

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

gdkpixbufdec: Fix pixbuf_loader leak during failures

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

rtpbin: Add missing break

rtpmanager: Take into account packet rate for max-dropout and max-misorder calculations

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

rtpmanager: add "max-dropout-time" and "max-misorder-time" props

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

qtmux: Fix date memory leak

When getting date from taglist, the memory should be freed after
using it.

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

qtmux: Fix sample memory leak

When getting sample from taglist, the memory should be freed after
using it.

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

cutter: Fix buffer leak

Buffer is added to the internal cache, and pushed only when accumulated
buffer duration crosses 200 ms. So when the chain ends, the buffer accumulated
is not freed. Freeing the cache when the state changes from PAUSED to READY.

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

rtpmux: Use default upstream event handling

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

rtpmux: As 0xFFFFFFFF is a valid ssrc, check if it has been set

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

gstrtpmux: allow the ssrc-property to decide ssrc on outgoing buffers

By not doing this, the muxer is not effectively a rtpmuxer, rather a
funnel, since it should be a single stream that exists the muxer.

If not specified, take the first ssrc seen on a sinkpad, allowing upstream
to decide ssrc in "passthrough" with only one sinkpad.

Also, let downstream ssrc overrule internal configured one

We hence has the following order for determining the ssrc used by
rtpmux:

0. Suggestion from GstRTPCollision event
1. Downstream caps
2. ssrc-Property
3. (First) upstream caps containing ssrc
4. Randomly generated

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

udpsrc: Fixup last commit

Update GLib dependency to 2.40.0

rtpstats: add utility for calculating RTP packet rate

qtdemux: handle empty segments in seeking adjust

If seeking targets an empty segment skip it as there is no media
offset to get from it. Instead look for the next one.

This doesn't make seeking in push-mode work if you seek to an
empty segment but at least won't get you to wrong offsets.

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

splitmuxsink: post messages when fragments are being opened and closed

This can be useful for applications that need to track the created fragments
(to log them in a recording database, for example)

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

splitmuxsink: allow non-video streams to serve as reference

In the absence of a video stream, the first stream will be used as
reference.

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

splitmuxsink: initialize mux_start_time properly

mux_start_time refers to the running_time of the buffer
that goes first in the output file. Normally this time is
0, so this variable is initialized to 0 during the state
change to PAUSED.

However, when dealing with dynamic pipelines and starting
a recording while the pipeline has already run for a while,
the running_time of the first buffer is > 0 and this causes
a problem with detecting the end of the first file(s) when
splitting by duration, because the code will later compare
the threshold_time with (last buffer running_time - mux_start_time)
and will get it wrong until mux_start_time advances enough
to make this difference < threshold_time, creating empty files
in the meantime.

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

avidemux: Reverse playback does not consider segment.start

During reverse playback, the media should stop playing at segment.start
This does not happen, and avidemux continues to process data even when
current timestamp is less that segment.start.

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

qtdemux: Check multi trex to find track id in mp4 mpeg-dash stream

If stream has more than one trex box which is not matched to actual
track id, it makes qtdemux crashed.

Author : Manasa Athreya (manasa.athreya@lge.com)

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

smpte: get size, stride info using VideoInfo

Use VideoInfo data to get size stride and
offset, instead of hard coded macros.

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

smpte: free mask

Free the memory allocated to 'mask' to avoid
memory leak.

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

gstreamer: good: tests: 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

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

rtpsource: doesn't handle probation and rtp gap in case of sender

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

rtpmanager: add new on-new-sender-ssrc, on-sender-ssrc-active signals

Allows for applications to get internal source's RTP statistics.
(eg. sender sources for a server/client)

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

ximagesrc: Gather and coalesce all damaged areas before retrieving.

These days the xserver seems to give us the same damage regions
over and over for entire windows, and we retrieve them multiple
times, which gives time for more damage to appear. Instead, just
quickly gather all damaged areas into a region list and copy
out once.

goom2k1: use the new audiovisualizer base class

Rebase to have goom using the GstAudioVisualizer base class in
gst-plugins-base/gst-libs/gst/pbutils

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

goom: use the new audiovisualizer base class

Rebase to have goom using the GstAudioVisualizer base class in
gst-plugins-base/gst-libs/gst/pbutils

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

deinterleave: implement accept-caps

Avoid using default accept-caps handler that will query downstream
and is more expensive. Just check if the caps is compatible with
the template and check if the channels are the same.

tests: deinterleave: also check for caps query results

deinterleave: use the caps query filter

It was being ignored and would lead to wrong results if the
element doing the query would rely on the intersection being made.

deinterleave: implement a caps query handler for the sinkpad

It was missing and apparently code relied on having it there
for not allowing a change in the number of channels

deinterleave: fix caps leak

Caps from the pad template are being leaked. In any case it is
from a static pad template and will 'leak' in the end, just doing
the cleanup for the good practice.

tests: gdkpixbufoverlay: add minimal unit test

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

gdkpixbufsink: don't leak old pixel buffer when setting a new overlay

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

flacenc: avoid potential string overflow

We don't necessarily have full control over the input tags, so
it's possible that the ISRC tag contains a longer string than
expected, in which case we'd write over the end of the static-size
13 byte buffer that is FLAC__StreamMetadata_CueSheet_Track::isrc.
Make sure to only copy the ISRC if it's not too long, and make
sure the buffer we write to is always NUL-terminated by using
g_strlcpy().

CID 1324931.

matroskademux: Remove leftover assertion from 0.10

We now allocate memory via GstAllocator and as such can handle arbitrary
alignments, not only <= G_MEM_ALIGN.

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

aacparse: fix uninitialized variables in LOAS config reading

On reading LOAS config, flag v=1 and vA=1 combination can occur, leading to warning
"Spec says "TBD"...". Returning TRUE on this case while parameters 'sample_rate' and
'channels' are pointing to uninitialized values can end on setting random values as
rate and channels on src caps.

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

Fix some compiler warnings when building with G_DISABLE_ASSERT

Touches rtpmanager and gdkpixbufsink

qtdemux: support timed-text subtitle tracks.

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

gst: Don't use deprecated gst_segment_to_position()

rtpbin/rtpjitterbuffer/rtspsrc: Add property to set maximum ms between RTCP SR RTP time and last observed RTP time

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

rtpbin/session: Allow RTCP sync to happen based on capture time or send time

Send time is the previous behaviour and the default, but there are use cases
where you want to synchronize based on the capture time.

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

Back to development

Release 1.6.0

Update .po files

smptealpha: Do not set width/height before comparing with old values

Otherwise we end up considering the values did not change and we wrongly
work with the old video format (which will lead to wrong
behaviour/segfaults).

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

qtdemux: Accumulate segments for edit lists before activating the next segment

eceb2ccc739092d964d78945e19c2ecedbd214e2 broke segment seeks by always
accumulating segments manually when activating a segment. This is only
needed when handling edit lists, not when activating a segment because of a
seek. Do the accumulation when switching edit list segments instead.

This fixes segment seeks again, while keeping edit lists playback working.

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

spectrum: send phase values in the GstMessage for Phase info

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

matroska-mux: Don't output a warning on MONO multiview mode.

rtptheoradepay: Fix memory leaks

The same memory leaks were fixed in identical fashion for
vorbisdepay in 06efeff5d979576a252e5dae57f46d6445b1df12 in 2009.

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

rtp{vorbis,theora}{pay,depay}: Cosmetic cleanup

* use g_list_free_full(), don't iterate elements maually when freeing
* call gst_rtp_*_pay_clear_packet(), don't duplicate its code
* use gst_buffer_unref() to clarify that it is buffers being released,
  instead of refering directly to gst_mini_object_unref()

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

rtp{vorbis,theora}pay: Store headers in the packet buffers lists, not a NULL buffer

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

Release 1.5.91

Update .po files

po: Update translations

avidemux: Fix taglist leak

gst_tag_list_insert() does not take ownership of the inserted taglist.

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

aacparse: Skip LOAS AAC until a valid config is seen.

It's normal when dropping into the middle of a stream to
not always have the config available immediately, so skip LOAS
until a valid config is seen without either setting invalid
caps or erroring out.

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

rtpjitterbuffer: reset just a bit more upon flush_stop

rtpjitterbuffer: remove dead struct member

multiudpsink: fix GError memory leak when hostname resolution fails

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

matroskamux: drop HEADER flag from output buffers

Drop HEADER flag from output buffers if they are not indeed
headers.

Fixes resending of headers in tcp connection handling

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

matroskamux: fix matroskamux ! matroskademux

Don't carry over DISCONT flags from the input buffers to the
output buffer, or the demuxer might reset its state when it
receives the first data buffer just after parsing the simple
block header, and then expect sane data to follow.
Fixes matroskamux ! demux erroring out.

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