matroska: read: fix tag list memory leak

gst_toc_entry_merge_tags makes a new ref of the taglist, so it should
be unref'ed as soon as the tags are merged to the tocentry

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

wavpackdec: fix taglist memory leak

When passing the taglist to gst_audio_decoder_merge_tags, the reference is increased
by audiodecoder and the caller should free the taglist being passed.

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

v4l2transform: fix pad closing

Signed-off-by: Jean-Michel Hautbois <jean-michel.hautbois@veo-labs.com>
https://bugzilla.gnome.org/show_bug.cgi?id=753875

Release 1.5.90

Update .po files

po: Update translations

multifilesrc: fix regression with starting from index set via index property

When we haven't started yet, set the start_index when we set the index property,
so that we start at the right index position after the initial seek. The index
property was never really meant to be for writing, but it used to work, so let's
support it for backwards compatibility.

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

qtdemux: fix offset calculation when parsing CENC aux info

Commit 7d7e54ce6863ff53e188d0276d2651b65082ffdb added support for
DASH common encryption, however commit
bb336840c0b0b02fa18dc4437ce0ded3d9142801 that went onto master
shortly before the CENC commit caused the calculation of the CENC
aux info offset to be incorrect.

The base_offset was being added if present, but if the base_offset
is relative to the start of the moof, the offset was being added twice.
The correct approach is to calculate the offset from the start of the
moof and use that offset when parsing the CENC aux info.

flacenc: actually return true for accept-caps query handling

rtp: copy metadata in the (de)payloaders which is missed before

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

directsoundsink: allow specifying audio playback device

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

flacenc: remove single entry if from loop

Iterate from the 2nd channel on and create the 1 channel struct
outside to make loop structure simpler and only slightly faster.

flacenc: implement proper accept-caps

Should just compare with what can be immediatelly accepted by
the element. flacenc can't renegotiate so if it has a caps already
it should only accept if it is that caps otherwise just use the
template caps

flacenc: improve sink pad template caps

Removes the need for custom caps query handling and makes it more
correct from the beginning on the template. It is a bit uglier
to read because there is 1 entry per channel but makes code easier
to maintain.

y4mencode: fix gst-launch version in documentation

audioencoders: use template subset check for accept-caps

It is faster than doing a query that propagates downstream and
should be enough

Elements: speexenc, wavpackenc, mulawenc, alawenc

videoencoders: use template subset check for accept-caps

It is faster than doing a query that propagates downstream and
should be enough

Elements: jpegenc, pngenc, vp8enc, vp9enc, y4menc

mpegaudioparse: use new baseparse API to fix tag handling

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

audioparsers: use new base parse API to fix tag handling

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

flacparse: use new baseparse API and fix tag handling

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

qtdemux: Use signed integer type to be able to check for negative subtraction results

CID 1315829

rtpvorbisdepay: remove dead code

payload_buffer must be NULL in ignore_reserved. Check will always be false.

Introduced by b1089fb5207697ba26edb4ff66ed0f465c6df3cf

CID #1316476

alawenc: port to AudioEncoder base class

audiodecoders: use default pad accept-caps handling

Avoids useless check of downstream caps when handling an
accept-caps query

Elements: flacdec, speexdec, wavpackdec, mulawdec, alawdec

videodecoders: use default pad accept-caps handling

Avoids useless check of downstream caps when handling an
accept-caps query

Elements: jpegdec, pngdec, vp8dec, vp9dec

alawdec: make error handling a bit nicer

Print the element along with the debug to make it easier to trace
the failures

alawdec: port to audiodecoder base class

mulawdec was already ported, alawdec was left behind.

qtdemux: only look for more samples in moofs in pull-mode

For playback of some fragmented formats with qtdemux it will
try to look for the next moof after finishing one but it is only
possible for pull-mode. For playback of streaming fragmented formats
such as DASH it should just not try to look for another moof but
instead wait for more data.

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

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

dcaparse: Don't look for a second syncword

There are streams out there that consistently contain garbage between
every frame so we never ever find a second consecutive syncword.

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

vp8enc, vp9enc: reset multipass file index when stopping encoder

Fixes multipass encoding when re-using the same element/pipeline
for subsequent encoding runs.

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

vp9enc: provide support for multiple pass cache files

Some files may provide different caps insight of one stream. Since
vp9enc support caps reinit, we should support cache reinit too.
If more then file cache file will be created, the naming will be:
cache cache.1 cache.2 ...

Based on patch by: Oleksij Rempel <linux@rempel-privat.de>

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

tests: aacparse: use caps query instead of accept-caps

The accept-caps query just does a shallow check at the current
element while at this test we want it to also look at downstream.
So use caps query there.

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

audioparsers: enable accept-template flag

Do a quick check with the pad template caps as it is enough. Users
should have figured the appropriate full caps on a previous caps query

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

rtspsrc: send the User-Agent header

Sometimes it is useful to know this information on the
server side. Other popular implementations (vlc, ffmpeg, ...)
also send this header on every message.

This includes a new "user-agent" property that the user
can set to use a custom User-Agent string. The default
is "GStreamer/<version>"

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

rtspsrc: wrap gst_rtsp_message_init_request in a local function

This will allow adding common request initialization, like the
user agent string, in just one place.

audioecho: make sure buffer gets reallocated if max_delay changes

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

vp8enc: provide support for multiple pass cache files

Some files may provide different caps insight of one stream. Since vp8enc
support caps reinit, we should support cache reinit too.
If more then file cache file will be created, the naming will be:
cache
cache.1
cache.2
...

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

rtpmp4gdepay: fix timestamps for RTP packets with multiple AUs

Use constantDuration to calculate the timestamp of non-first AU in the
RTP packet.

If constantDuration is not present in the MIME parameters, its value
must be calculated based on the timing information from two consecutive
RTP packets with AU-Index equal to 0.

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

souphttpsrc: remove unnecessary if, g_free is null safe

souphttpsrc: add property to set HTTP method

To allow souphttpsrc to be use HTTP methods other than GET
(e.g. HEAD), add a "method" property that is a string. If this
property is not set, GET is used.

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

check: Rename states unit test

Makes it easier to differentiate from other modules states unit test

goom: Rename get_type() function of base class to prevent symbol conflicts

This is a problem when statically linking.

rtpjitterbuffer: Keep the DTS estimate if we got no DTS after a jitterbuffer reset

Otherwise we will just output buffers without timestamps after a reset if no
timestamps are provided by upstream, e.g. when using RTSP over TCP.

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

matroska: Remove unused variable

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

rtp: Copy metadata in the (de)payloader, but only the relevant ones

The payloader didn't copy anything so far, the depayloader copied every
possible meta. Let's make it consistent and just copy all metas without
tags or with only the video tag.

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

qtdemux: fix small typo in comment

goom2k1/doc: Fixup previous commit

goom2k1/doc: Use GstGoom2k1 namespace

The doc generator isn't happy when we have class name clash. Simply
use it's own namespace.

audioecho: removed unused variable in set_property

unused local variable 'delay' is removed.

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

qtdemux: fix suboptimal queue iteration code

qtdemux: don't use glib 2.44-only API

qtdemux: add support for ISOBMFF Common Encryption

This commit adds support for ISOBMFF Common Encryption (cenc), as
defined in ISO/IEC 23001-7. It uses a GstProtection event to
pass the contents of PSSH boxes to downstream decryptor elements
and attached GstProtectionMeta to each sample.

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

rtph264depay: checking if depay has sps/pps nals before insertion

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

matroskamux: fix outdated comment

The default behaviour was changed in the 0.10 -> 1.x
transition, but the comment was not updated.

rtptheorapay: If flushing a packet failed, go out of the loop immediately

rtpvorbispay: If flushing a packet failed, go out of the loop immediately

rtptheorapay: Extract pixel format from the ident header to put it into the sampling field of the caps

We always put 4:2:0 into the caps before, which obviously is wrong for 4:2:2
and 4:4:4 formats.

rtpklv(de)pay: add "RTP" in the klass string

GstRTSPMedia uses this classification to detect the real payloader
inside a dynpay bin and asserts if it doesn't find it, therefore
it is required

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

tests: rtpaux: use a dynamic pt in the test

1) Tests that using dynamic PT instead of the default ones work
2) If we ever decide to change the codec here we don't need to
   worry about change the PT for the default one of the new codec
   in the test

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

rtprtxsend: print valid type where guint32 is expected

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

rtppayload: set standard payload type as default

Initialize the PT to the default value of the codec and check if
it is still the default before declaring the pt to be dynamic or
not when setting the caps.

Also use the PT constants from the rtp lib when possible

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

qtdemux: store the moof-offset also for push mode

It will be used in some cases for getting the correct offsets
from trun atoms.

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

qtdemux: handle default-base-is-moof flag

Handle the flag from the tfhd that signals the base offset to
start from the moof atom

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

matroskademux: Preserve forward referenced track tags

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

tests: rtpaux: fix test failure

The RTP PT for alaw is 8.
Less than 50 packets are received in the length of this test so it
would never drop a buffer or would drop only the last buffer and
it would fail sometimes when the received wouldn't receive the
retransmission packet in time.

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

rtpstreamdepay: Only allow activation in push mode

We need a proper caps event from upstream with the full RTP caps as we can't
create caps ourselves from thin air. Fixes usage of rtpstreamdepay after e.g.
a filesrc or any other element that supports pull mode.

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

soup: fix typo in translated string

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

rtph264depay: Put the profile and level into the caps

rtph264depay: Only update the srcpad caps if something else than the codec_data changed

h264parse does the same, let's keep the behaviour consistent. As we now
include the codec_data inside the stream too here, this causes less caps
renegotiation.

rtph264depay: PPS replaces and old PPS if it has the same id, independent of SPS id

The spec says:

When a picture parameter set NAL unit with a particular value of
pic_parameter_set_id is received, its content replaces the content of the
previous picture parameter set NAL unit, in decoding order, with the same
value of pic_parameter_set_id (when a previous picture parameter set NAL unit
with the same value of pic_parameter_set_id was present in the bitstream).

splitmuxsink: remove extra \n at debug message

splitmuxsink: prevent deadlock when states change too fast

If the GOP is completed, pads have to start gathering for the
next one but it is possible that the the state might go to
COLLECTING_GOP_START and back to WAITING_GOP_COMPLETE before the
thread has a chance to wake up and proceed, leaving it trapped in
the check_completed_gop loop and deadlocking the other threads
waiting for it to advance.

To solve it, this patch also checks that tha input running time
hasn't changed to prevent this scenario.

rtph264depay: Insert SPS/PPS NALs into the stream

h264parse does the same and this fixes decoding of some streams with 32 SPS
(or 256 PPS). It is allowed to have SPS ID 0 to 31 (or PPS ID 0 to 255), but
the field in the codec_data for the number of SPS or PPS is only 5 (or 8) bit.
As such, 32 SPS (or 256 PPS) are interpreted as 0 everywhere.

This looks like a mistake in the part of the spec about the codec_data.

souphttpsrc: handle empty http proxy string

1) If the system http_proxy environment variable is not set
or set to an empty string, we must not set proxy to avoid
http connection error.

2) In case of proxy property setting, if user want to clear
the proxy setting, they should be able to set it to NULL or
an empty string again, so this is fixed too.

3) Check if the proxy string was parsed correctly.

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

dvdemux: remove unused variable

Remove unused variable 'framecount' from dvdemux

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

rtspsrc: assertion error due to wrong condition check

In media to caps function, reserved_keys array is being used for variable i,
leading to GLib-CRITICAL **: g_ascii_strcasecmp: assertion 's1 != NULL' failed
changed it to variable j

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

rtpmp4vdepay: rtpbuffer is being unref'ed twice

process_rtp_packet doesn't transfer the rtp buffer to mp4v_process_depay
the refernce should not be removed here

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

rtspsrc: Strip keys from the fmtp that we use internally in our caps

Skip keys from the fmtp, which we already use ourselves for the
caps. Some software is adding random things like clock-rate into
the fmtp, and we would otherwise here set a string-typed clock-rate
in the caps... and thus fail to create valid RTP caps

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

splitmuxsink: Support mpegtsmux as a muxer.

As a fallback, look for a pad template sink_%d on
the muxer when requesting pads, to support mpegtsmux

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

splitmuxsrc: Use a separate lock to delay typefind.

Don't hold the main splitmux part lock over
the parent state change function, as it prevents
posting error messages that happen. Since the purpose
is to prevent typefinding from proceeding, use a
separate mutex just for that.

matroska: fix memory leak

After adding to tag list, key_val is not being free'd
resulting in memory leak

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

qtdemux: fix 16-bit PCM audio advertised with 'raw ' fourcc

'NONE' and 'raw ' fourcc don't always contain U8 audio, it can
be more bits as well, in which case it's just like 'twos'.

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

v4l2: Allow framerate to be large then 100pfs

This limit was arbitrary. We still fixate near 100pfs for compatibility.

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

avidemux: Stop without posting error on flushing

This could just be a normal pipeline shutdown.

v4l2bufferpool: set GST_BUFFER_COPY_FLAGS to copy flags also

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

tests: add minmal matroskademux test for subtitle output

Some of the subtitle chunks will have embedded
NUL-terminators (last three), some don't (first three),
some will have markup, some won't, some will be valid
UTF-8 (all but last), some won't (last stanza).

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

matroskademux: fix for subtitle buffers with NUL terminators

Commit 45892ec8 created a regression where g_utf8_validate() would fail
if the subtitle buffer had a NUL terminator as part of the data.

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

rtpvp8depay: Check available bytes before copy

Need to check that the number of bytes we want to copy from the adapter
actually is available and handle the error case gracefully. This error
may happen if malformed packets are received and we don't have a
complete frame.

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

qtdemux: Support subtitle when track subtype is fourcc_subt

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

v4l2bufferpool: Set timestamp when queue buffer.

Should set timestamp when queue buffer.

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

rtpmux: handle different ssrc's on sinkpads

Do this by not putting the ssrc from the src pads in the caps used to
probe other sinkpads, and then  intersecting with it later.

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

Update mailing list address from sourceforge to freedesktop

matroskademux: fix trailing '*' displayed with some text subtitles

The subtitle buffer we push out should not include a NUL terminator
as part of the data, we just add such a terminator for safety, but
it should not be included in the buffer size.

A NUL terminator is not valid UTF-8, so checks will fail if it's
included in the size, and the NUL will be replaced by the fallback
character specified when converting, i.e. '*'.

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

pulse: add properties to GstDevice

Add the extra properties we get from pulse to the GstDevice we expose
with the device monitor

audiofx: Fix typo in example pipelines

Fix typo in example pipelines of audiowsincband and audioinvert.

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

splitmuxsink: add a "format-location" signal that allows better control over filenames

In certain applications, splitting into files named after a base
location template and an incremental sequence number is not enough.

This signal gives more fine-grained control to the application to
decide how to name the files.

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

osxaudiosrc: no resampling on OS X

Unlike Remote IO, AUHAL doesn't have built-in resampling
for sources -- confirmed by Core Audio engineer Doug Wyatt:
http://lists.apple.com/archives/coreaudio-api/2006/Sep/msg00088.html

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

osxaudiosrc: avoid get_channel_layout

This only produces a warning and serves no purpose.

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

osxaudio: Avoid making a duplicate structure in caps for mono/stereo case

For 1ch or 2ch devices, we just need to set the caps to allow both
options since CoreAudio will up/downmix appropriately.

Also fixes the condition for the 2ch case to be exact, rather than at
least 2 channels since the downmix will not take place in the >stereo
case.

osxaudio: Don't set the format on an initialized AudioUnit

We need to initialize the AudioUnit early to be able to probe the
underlying device, but according to the AudioUnitInitialize() and
AudioUnitUninitialize() documentation, format changes should be done
while the AudioUnit is uninitialized. So we explicitly uninitialize the
AudioUnit during a format change and reinitialize it when we're done.