isomp4: Remove an extra bracket in a comment.

splitmuxsrc: Protect total_duration state variable with the object lock.

Prevent deadlocks from downstream querying duration from the streaming thread.

Automatic update of common submodule

From 21ba2e5 to d37af32

goom: clean dereferences of private structure

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

goom2k1: clean dereferences of private structure

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

Automatic update of common submodule

From c408583 to 21ba2e5

docs: remove variables that we define in the snippet from common

This is syncing our Makefile.am with upstream gtkdoc.

Automatic update of common submodule

From d676993 to c408583

Back to development

Release 1.5.1

Update .po files

rtpsession: Override the SSRC from the packets' SSRC if none was given via caps or property

po: Update translations

v4l2: Don't warn when optional CID are not implement

gst_v4l2_get_attributre() shall only be used when the CID is expected
to be supported. Otherwise, we get unwanted warning posted to the bus.

rtpsession: Only suggest our internal ssrc if it's not a random one and was selected as internal ssrc

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

interleave: error when channel-positions-from-input=False

self->channels is being incremented only when
channel-positions-from-input is set as TRUE. So in case of FALSE
self->func is not set and hence creating assertion error.
Hence removing the condition to increment self->channels.

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

rtpjitterbuffer: Add support for receiving reduced size RTCP

It worked before but gave warnings, now we just ignore RTCP
packets that don't start with a SR. As all we're interested
in here are SRs.

rtpssrcdemux: Add support for reduce size rtcp

According to RFC 5506, reduce size packages can be sent, this
packages may not be compound, so we need to add support for
getting ssrc from other types of packages.

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

rtpsession: Add support for receiving reduced size rtcp

See RFC 5506

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

aacparse: Add support for channel configurations 11, 12 and 14 and 7 actually has 8 channels

ISO/IEC 14496-3:2009/PDAM 4 added 11, 12 and 14.

asteriskh263: Un-rank clashing depayloader

This depayloader clash with the standard one for H263p. It produces an
H263p stream with a modified header. It uses encoding-name that is the
same as H263p (H263-1998) though the resulting ES is not decodable or
parsable in GStreamer, making it unsuable in dynamic pipeline. This
patch unrank this specialized depayloader since it can only be used in
custom pipeline.

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

goom2k1: remove variables not needed anymore

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

goom2k1: rebase to use the audiovisualizer class

Rebase to have goom2k1 using the common GstAudioVisualizer class

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

goom: rebase to use the audiovisualizer class

check: Use GST_CHECK_MAIN () macro everywhere

Makes source code smaller, and ensures we go through common initialization
path (like the one that sets up XML unit test output ...)

rtpsession: Only schedule a timer when we actually have to send RTCP

Otherwise we will have 10s-100s of thread wakeups in feedback profiles, create
RTCP packets, etc. just to suppress them in 99% of the cases (i.e. if no
feedback is actually pending and no regular RTCP has to be sent).

This improves CPU usage and battery life quite a lot.

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

rtpsession: Remove useless goto

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

examples: Set RTP profile to AVPF for rtpaux examples

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

rtspsrc: Set RTP profile on the rtpsession objects

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

rtpbin: Add rtp-profile property for setting the default profile of newly created sessions

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

rtpsession: Only put RRs and full SDES into regular RTCP packets

If we may suppress the packet due to the rules of RFC4585 (i.e. when
below the t-rr-int), we can send a smaller RTCP packet without RRs
and full SDES. In theory we could even send a minimal RTCP packet
according to RFC5506, but we don't support that yet.

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

rtpsession: Keep track of tp/tn and t_rr_last separately

Otherwise we can't properly schedule RTCP in feedback profiles as we need to
distinguish the time when we last checked for sending RTCP (tp) but might have
suppressed it, and the time when we last actually sent a non-early RTCP
packet.

This together with the other changes should now properly implement RTCP
scheduling according to RFC4585, and especially allow us to send feedback
packets a lot if needed but only send regular RTCP packets every once in a
while.

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

rtpsession: Add property for selecting RTP profile (AVP/AVPF/etc)

And modify our RTCP scheduling algorithm accordingly. We now can send more
RTCP packets if needed for feedback, but will throttle full RTCP packets by
rtcp-min-interval (t-rr-int from RFC4585).

In non-feedback mode, rtcp-min-interval is Tmin from RFC3550, which is
statically set to 1s or 0s by RFC4585. Tmin defines how often we should
send RTCP packets at most.

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

mulawdec: Let baseclass estimate bitrate

This makes playback directly from a file work with the right caps.

dynudpsink: keep GCancellable fd around instead of re-creating it constantly

And create it only when starting the element.

udpsink, multiudpsink: keep GCancellable fd around instead of re-creating it constantly

Otherwise we constantly create/close event file descriptors,
every time we call g_socket_condition_timed_wait() or
g_socket_send_message(s)(), i.e. a lot. Which is not
particularly good for performance.

Can't create GCancellable in ::start() here because it's used
in client_new() which may be called via the add-client action
signal which may be called before the element is up and running.

udpsrc: keep GCancellable fd around instead of re-creating it constantly

Otherwise we constantly create/close event file descriptors,
every single time we call g_socket_condition_timed_wait() or
g_socket_receive_message(), i.e. twice per packet received!
This was not particularly good for performance.

Also only create GCancellable on start-up.

matroska: overwritten value assignment

curpos is set and immediately after, set again. Remove the redundant
assignment.

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

rtpvrawdepay: don't shadow existing outbuf variable

And fix unref of the wrong one which will contain NULL
in an error code path.

rtpvrawdepay: map/unmap output frame only once, not for every input packet

Map output buffer after creating it and keep it mapped
until we're done with it instead of mapping/unmapping
it for every single input buffer.

qtdemux: remove fixme from 2006

It has been verified by use over time.

qtdemux: fix reverse playback of fragmented media

qtdemux creates a samples array and gets the timestamps for buffers by
accumulating their durations. When doing reverse playback of fragments,
accumulating samples will lead to wrong timestamps as the timestamps
should go decreasing from fragment to fragment and the accumulation
will produce wrong results.

In this case, when receiving a discont for fragmented reverse playback,
the previous samples information should be flushed before new data
is processed.

splitfilesrc: Implement binary search in find_part_for_offset

Implement binary search using gst_util_array_binary_search

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

rtpsession: Don't crash if we receive FIR/PLI from a source we don't know

rtpsession: Fix collection of statistics

Stats should be collected on the media rtp source not in the
sender one.

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

multifilesink: Add a new max-duration file switching mode

This new mode ensures that files will never exceed a certain duration
based on incoming buffer PTS (and duration if present)

Note:
* You need timestamped buffers (duh). If some of the incoming buffers don't
  have PTS, then it will just accept them in the current file

multifilesink: streamline the file-switch code a bit

Use the same functions regardless of the mode we are using

multifilesink: add "aggregate-gops" property to process GOPs as a whole

This property can be used in combination with next-file=max-size
(and perhaps a future next-file=max-duration) to make sure that
each file part starts cleanly with a key frame and the appropriate headers.

In order for this property to work correctly, upstream elements should make
sure than any headers that need to be written in a standalone file are:
1) in the streamheader caps field
2) and/or in the stream as one or more buffers marked with GST_BUFFER_FLAG_HEADER
   that are just before the keyframe buffer

This is useful for MPEG-TS/MPEG-PS file segmenting in
combination with mpegtsmux or mpegpsmux.

Original patch by: Tim-Philipp Müller <tim@centricular.com>

rtspsrc: Use single-include header for the RTSP library

udp: don't use soon-to-be-deprecated g_cancellable_reset()

From the API documentation: "Note that it is generally not
a good idea to reuse an existing cancellable for more
operations after it has been cancelled once, as this
function might tempt you to do. The recommended practice
is to drop the reference to a cancellable after cancelling
it, and let it die with the outstanding async operations.
You should create a fresh cancellable for further async
operations."

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

Revert "doc: Workaround gtkdoc issue"

This reverts commit 1797c8f8b12d7f4c7a9444c94f34f4d08ec85945.

This is fixed by the gtk-doc 1.23 release.
<para> cannot contain <refsect2>:
http://www.docbook.org/tdg/en/html/para.html
http://www.docbook.org/tdg/en/html/refsect2.html

rtpg726pay: fix caps leak

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

rtpg726depay: don't leak input buffer

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

rtpsource: Queue bad packets instead of dropping them

So we can send them out once we found the next, consecutive sequence number in
case one is following.

rtpsource: Use g_queue_foreach() to unref all buffers in queues

rtpsource: Refactor seqnum comparison code a bit

rtpsource: Allow sequence number wraparound during probation

rtpsource: Make sequence number comparison code more readable

... by using gst_rtp_buffer_compare_seqnum() and signed integers
instead of implictly using effects of integer over/underflows.

rtpjitterbuffer: When detecting a huge seqnum gap, wait for 5 consecutive packets before resetting everything

It might just be a late retransmission or spurious packet from elsewhere, but
resetting everything would mean that we will cause a noticeable hickup. Let's
get some confidence first that the sequence numbers changed for whatever
reason.

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

doc: Workaround gtkdoc issue

With gtkdoc 1.22, the XML generator fails when a itemizedlist is
followed by a refsect2. Workaround the issue by wrapping the
refsect2 into para.

qtdemux: avoid wrong warnings on unknown node types

Add 'name' and 'mean' fourccs, as we handle them. Right now each use would
trigger a warning.

rtpg726depay: add block_align to output caps

It is needed to correctly negotiate caps with matroskamux
and most other muxers.

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

audiofxbasefirfilter: Fix time-domain convolution with >1 channels

input_samples is the number of frames, but we used it as the number of
samples.

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

vp[89]enc: Properly convert between GStreamer and encoder timebase

... by switching numerator and denominator when scaling.

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

vp[89]enc: Don't set timebase from the framerate

The framerate very often is just an indication of the ideal framerate, not the
actual framerate of the stream. By just using the framerate, we confuse the
rate control algorithm algorithm as multiple frames will map to the same PTS
or have durations of 0.

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

tests: wavpackparse: fix unit test

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

docs: update example pipelines in element docs

Mostly gst-launch -> gst-launch-1.0
Use autovideosink/autoaudiosink more often.
Sprinkle some converters here and there.

po: update POTFILES.in

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

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