splitmuxsrc: minor error message clean-up

Don't put filename in error message shown to user.

flacparse: fix buffer leak when stored to seektable

Fix a leak with the
validate.file.playback.change_state_intensive.samples_multimedia_cx_flac_Yesterday_flac
scenario.

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

qtdemux: fix example pipeline in docs

The gst-launch script for example launch line to test qtdemux is
missing a queue before the decodebins, otherwise the gst-launch-1.0
command won't work.

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

Revert "rtpsession: Also report internal sources in on-new-ssrc and on-ssrc-active"

This reverts commit d22ec496328e6ba8edbf2d071d5608b2af2831e8.

Application code might expect that it only gets external sources on those
signals, and get confused by this. If anything we would need to add new
signals.

rtpsession: Also report internal sources in on-new-ssrc and on-ssrc-active

Without this it seems impossible for an application to easily get notified
about the internal ssrcs that are created, e.g. sender sources, and also
to know when they are active and produce RTCP packets.

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

jpegdec: fix frame leaks in handle_frame() implementation

handle_frame() is supposed to consume @frame, so if we don't call
gst_video_decoder_drop_frame() or gst_video_decoder_finish_frame() we have to
release it manually.

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

rtspsrc: Fix up last commit

rtspsrc: Only do RTX when using a feedback profile

rtpsession: The stats min_interval is in seconds, not nanoseconds

We have to scale it to compare it against our clock times.

rtpsession: Only return TRUE if early feedback was requested already and it's early enough

matroska: remove unused property enum items

qtdemux: fix buffer leak on eos in push mode

Based on patch by Guillaume Desmottes.

scenario: validate.http.playback.seek_with_stop.raw_h264_1_mp4

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

qtdemux: Check for sizes of the rdrf (redirect) atom before accessing the data and use g_strndup() instead of g_strdup()

Thanks to Ralph Giles for reporting this.

rtspsrc: Only enable retransmissions if there is retransmission info in the SDP

Otherwise we're going to send early RTCP and NACKs in non-feedback sessions
too, which will confuse servers.

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

dvdemux: extract recording time

Extracts the recorded time of the dv file from
the metadata and puts it into the global tags.

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

matroskademux: fix seek event leak

gst_matroska_demux_handle_seek_event() doesn't consume the
event so we have to unref it.

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

matroska-demux: Send pending tags when adding a new pad

We might've parsed those tags before already and tried to push them to
non-existing pads before. Now let's do it for real.

rtpstats: Average RTCP packet size is in bytes, bandwidths in bits

We need to convert the size to bits for our calculations.

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

rtpstats: Use the same lower limit for RTCP bandwidth to stop sending RTCP everywhere

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

rtpsession: Use bandwidth calculation by default instead of some arbitrary hardcoded value

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

rtpsession: Bandwidth is supposed to be in bits/s, not bytes/s

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

rtpjitterbuffer: Fix RTX unit test

The calculations were a bit off everywhere, even before the changes done
recently to the delay for RTX of expected future packets. It only worked by
accident, but now the calculations are all correct again. Hopefully.

Rename property enums from ARG_ to PROP_

Property enum items should be named PROP_ for consistency and readability.

rtpjitterbuffer: Fix "stats" property docs

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

Remove obsolete Android build cruft

This is not needed any longer.

videocrop: print the property values when set

Instead of printing the currently used values. The log is meant
to show what the properties changed to, not what is being currently
used.

remove unused enum items PROP_LAST

This were probably added to the enums due to cargo cult programming and are
unused. Removing them.

level: fix infinite loop for very low interval values

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

tests: define GST_CHECK_TEST_ENVIRONMENT_BEACON

Make sure the test environment is set up.

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

configure: bump automake requirement to 1.14 and autoconf to 2.69

This is only required for builds from git, people can still
build tarballs if they only have older autotools.

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

Update .gitignore

rtspsrc: Fix RTCP caps leak

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

rtpjitterbuffer: When request retransmissions for future packets, consider the packet spacing in the extra delay

We now take the maximum of 2*jitter and 0.5*packet_spacing for the extra
delay. If jitter is very low, this should prevent unnecessary retransmission
requests to some degree.

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

rtpjitterbuffer: Take a running average of the packet spacings instead of just the latest

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

rtpjitterbuffer: Add "rtx-next-seqnum" property

If this is set to FALSE, rtpjitterbuffer will not request retransmissions for
future packets based on when they are estimated to arrive.

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

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

rtxreceive: Put debug output for retransmission requests at the right place

Before it was only ever printed once for every time a ssrc was associated with
a specific stream.

v4l2: don't add the same interlace mode twice

Some drivers modify the interlace mode to progressive, no matter what
input you give them, make sure that we don't add the same interlace mode
twice.

equalizer: fix dynamic changes on bands

When we are in passthrough, the transform function doesn't run and if the
passthrough check is in this function it will never be deactivated. Fix this by
checking directly whenever a gain is changed.

Also set the passthrough to TRUE at init because the gains default to 0, so we
can passthrough until any gain property is changed.

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

Remove INSTALL file

autotools automatically generate this, and when using different versions
for autogen.sh there will always be changes to a file tracked by git.

Remove LICENSE_readme

It's completely outdated and just confusing, better if people are
forced to look at the actual code in question than trusting this file.

v4l2: cast unused return to void

Quell unchecked return value defect by casting the return value to void and
making it explicit it is going to be ignored.

CID #206031

vp8dec: optimize vpx image to gstbuffer copy when strides match

Solving this FIXME. Copy the full plane when strides are the same

vp9dec: optimize vpx image to gstbuffer copy when strides match

Solving this FIXME. Copy the full plane when strides are the same

ac3parse: fix memory leak

icydemux: Fix segfault if metadata-interval is 0

Prevents an extra unref of GstBuffer when passing a non-icy stream through
icydemux with metadata-interval set to 0.

Reproducible with:
gst-launch-1.0 filesrc location=~/testsong.mp3 ! \
'application/x-icy,metadata-interval=(int)0' ! icydemux ! decodebin ! wavenc ! \
filesink location=~/testsong.wav

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

audiofx: fix typo in example pipelines

Fix typo in example pipelines

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

osxaudio: fix spelling in debug message

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

tests: selectable amount of bands in equalizer demo

Adding an option in the equalizer demo to make the number of bands selectable.

rtpsource/rtprtxsend: Also pass correct seqnum-offset and payload to the RTX rtpsource

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

rtpsession: Track RTX ssrc caps

This is needed so that we can generate SR for RTX stream correctly (the
clock rate is required).

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

rtprtxsend: Copy over timestamps from the orignal buffers to the RTX buffers

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

tests: switch equalizer demo to play from uri

Switch the equalizer-nbands demo to use uridecodebin, so users can listen to
something more pleasant than white noise. If anybody misses the white noise
a uri handler to audiotestsrc can be used.

tests: improve readability of equalizer demo

Rename variable name to make it more readable, add comments for the three
scales created per block, and set the window title.

tests: add missing license header for equalizer demo

qtdemux: fix tag list leaks on error paths

qtdemux: fix tag list leak on unknown stream type

suppressions: ignore an apparent bug in strtod

A buffer overread.

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

splitmuxsink: do not access property variable without the object lock, use the local stack copy instead

splitmuxsink: add probe on the multiqueue's sink pad instead of the ghost pad

because _release_pad tries to release it from ctx->sinkpad, which is
multiqueue's sink pad, and currently fails because the probe is not
installed there

rtprtx*: Fix typos

rtpsession: Not sending early RTCP now because of dithering means we send it with the next compound packet

rtpsession: Improve debug output a bit if we can't allow early feedback

rtpvp8depay: When dropping intra packet, request keyframe

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

rtpjitterbuffer: Change resyncing GST_WARNING to GST_INFO

This also happens in the very beginning when we receive the first packet, a
warning would be very confusing here. In all places where we should warn about
this, we would've printed a warning already before.

multifilesink: minor docs improvement

rtpjitterbuffer: Add "rtx-max-retries" property

This property allows to limit the maximum number of retransmission
for a specific packet.

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

rtpjitterbuffer: Fix expected_dts calc in calculate_expected

Right above we consider lost_packet packets, each of them having duration,
as lost and triggered their timers immediately. Below we use expected_dts
to schedule retransmission or schedule lost timers for the packets that
come after expected_dts.

As we just triggered lost_packets packets as lost, there's no point in
scheduling new timers for them and we can just skip over all lost packets.

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

rtpjitterbuffer: Make the next output buffer discont after resetting the jitterbuffer

Resetting the jitterbuffer drops all packets and other things, and will cause
a discontinuity in the packets received by the depayloaders. They should now
also flush anything they had pending as the new data will start at a different
position.

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

qtdemux: Update segment.start after key-unit seek

When doing key uint seek, qtdemux calls gst_qtdemux_adjust_seek
to get proper offset. And then this offset is set to
segment.position and segment.time in gst_qtdemux_perform_seek but
segment.start is not updated.

After that, application sends segment query,
qtdemux sets start and stop to query using gst_segment_to_stream_time. Due
to the wrong value in segment.start, the stop position is smaller than
it should.

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

qtmux: remove useless variable do_pts

We always write the CTTS in qtmux. Ideally we only want to do that
for streams that need DTS, it should be present on the track information
rather than be decided based on each buffer

qtmux: remove subtraction that makes PTS/DTS start from 0

As qt uses durations, it doesn't matter, only the difference
between consecutive buffers is important. Also, collectpads
already replaces PTS/DTS with the running times for them.

tests: qtmux: add tests to verify it handles non-0 segments

Both input streams in this test have a segment.start = 10s, so
output should start from 0 anyway.

Another test has both starting at non-0 segments, but the running
time of both streams should still start from 0

tests: qtmux: simple muxing test

Adds a new simple test that verifies that data is properly muxed
and preserved.  PTS, DTS, duration and caps are verified.

smpte: remove unused fields

Remove the fields - format and fps from smpte
as they are unused.

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

tests: add test suite for alpha

Added test suite for alpha element with test cases
1. alpha
2. chroma keying

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

suppressions: add a well known zlib inflate bug

splitmuxsink: fix mutex leak

tests: Fix rtprtx test by handling buffer lists

Commit #1018aa made rtprtxsend handle buffer lists, breaking
the test which probes for buffers, but not buffer lists.

Use a utility function to run the probe callback on each buffer
in the list in turn and remove any buffers that are dropped.

isomp4: Refactor various state variables into a mux_mode var

Instead of checking various state variables around the muxer,
track the current muxing mode in a single 'mux_mode' enum.

Add some implementation notes about the different mux modes

tests: Use AM_TESTS_ENVIRONMENT

Needed by the new automake test runner

rtph263depay: Fix framesize parsing

The string passed to the parsing function only contains a framesize, and
not <pt> + <framesize>

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

wavparse: clip chunk size above the valid maximum (0x7fffffff)

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

wavparse: clip chunk length to available data (when known)

This prevents silly chunk lengths from possibly overflowing
(at least when we know the actual data length).

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

qtdemux: Don't accumulate segment bases manually

gst_segment_do_seek() does that for us already, and doing it twice
will break non-flushing seeks in interesting ways. Leftover from 1.0
porting.

Also copy over segment offset and applied_rate, just in case.

icles: Fix waiting for segment-done if it happens too fast

Sometimes we can get segment-done before we got async-done. If we waited
for async-done only, the segment-done would be dropped and we would wait
forever for it a few lines below.

qtdemux: stbl_index is valid from 0 onwards

It indicates the last sample parsed, not the next one to parse.
As it starts in -1, any value from 0 onwards means that it has
some valid data.

docs: make GstRTCPSync enum show up in rtpbin docs

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

docs: add RTPJitterBufferMode enum to rtpbin docs

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

multifilesink: close files before posting message

Makes sure the files were properly flushed and closed before
the message reaches the application

tests: multifile: increment tests to check for multifile messages

Also verify that the multifilesink file messages are being correctly
posted to the bus

tests: multifile: handle FIXME for proper checking when test finished

Use a GstBus and wait for EOS to finish the tests instead of
relying on sleeping

multifilesink: post file message on EOS

When multifilesink is operating in any mode other than one file
per buffer, the last file created won't have a file message posted
as multifilesink doesn't handle the EOS event.

This patch fixes it by using the last position to post a file
message when EOS is received. This should ensure at least the
time related data and the filename are posted to the application
or other elements

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

Automatic update of common submodule

From bc76a8b to c8fb372

qtdemux: Guard against 64-bit overflow

For large-file atoms, guard against overflow in the size field,
which could make us jump backward in the file and cause
infinite loops.

isomp4: Make non-seekable downstream an error in normal mode

When not in fast-start or fragmented mode, we need to be able
to rewrite the size of the mdat atom, or else the output just
won't be playable - the mdat placeholder with size == 0 will
cover the rest of the file, including any moov atom we write out.

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

rtph263pay/-depay: add framesize SDP attribute

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

rtpjpegpay/-depay: Remove incorrectly introduced framesize SDP attribute

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

v4l2src: device sequence/offset correction in case of renegotiation

The v4l2 device restarts the sequence counter in case of streamoff/streamon,
the GST offset values are supposed to increment strictly monotonic, so
adjust the sequence counter/offset values in case of caps
renegotiation.

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

v4l2src: add frame loss detection

In case of v4l2 driver filled offset/sequence values add frame
loss detection (and write a warning message).

Move offset meta data setting and frame loss checking after the
timestamp adjustment code to get proper timestamps for the
warning message.

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

v4l2: use v4l2 capture device sequence counter

Use the v4l2 capture device sequence counter for
setting the GstBuffer offset/offset_end values.

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