audioparsers: Improve src template caps

Remove the parsed/framed fields and add all fields to the template
caps that always exist.

aacparse: parse codec_data to determine number of samples per frame

Fixes #656734.

Automatic update of common submodule

From a39eb83 to 11f0cd5

Automatic update of common submodule

From 605cd9a to a39eb83

matroskamux: make default duration check less sensitive

Frame duration might vary for 1 usecond, in this case matroskamux
decides to create BLOCKGROUP instead of SIMPLEBLOCK.

Convert duration to timecodescale which is (typically) less precise, and
then also allow the difference of 1/-1 to arrange for less sensitive check.

Based on patch by Alexey Fisher <bug-track@fisher-privat.net>

Fixes #653080.

rtpmp4gdepay: improve bogus interleaved index compensating

Patch by <gudake@gmail.com>

Fixes #654585.

souphttpsrc: Allow positive, non-1.0 segment rates

Only negative rates are not supported. Fixes bug #658305.

tests: parsers: provide more real data when testing draining of garbage

amrparse: fix and streamline valid frame checking

... to handle various combinations of sync or not, and sufficient data
or not as might be expected.

Fixes #650714.

qtdemux: fragmented support; avoid adjustment for keyframe seek

... since all index data may not yet be available at that time.

qtdemux: fragmented support; mark all audio track samples as keyframe

qtdemux: fragmented support; properly init return variable value

Fixes #655918.

rtspsrc: add gtk-doc for new short-header property

rtspsrc: allow sending short RTSP requests to a server

Some encoders (Arecont) do not like the long OPTIONS sent at startup as sent by
GStreamer, but do accept the short header as sent by Live555.

This patch makes the extending the request optional by adding a property
(short-header).

Fixes #655805.

API: GstRTSPSrc:short-header

rtph263ppay: Set H263-2000 if thats what the other side wants

The static caps states this element supports H263-2000, but setcaps never
sets it, so it was lie.

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

rtpsession: Initialise the last_keyframe_request variable

multiudpsink: make add/remove/clear/get-stats action signals

http://bugzilla.gnome.org/show_bug.cgi?id=657830

Signed-off-by: Peter Korsgaard <jacmet@sunsite.dk>

qtdemux: push mode; perform some extra checks prior to upstream seeking

qtdemux: push mode; fix buffered streaming

That is, in case where no seek is peformed to moov, but preceding
limited mdat is buffered.

qtdemux: avoid overflow wraparound in timestamp when adding durations

Do some type juggling to avoid overflow, while still allowing for 'negative'
durations (which would need a wraparound effect).

v4l2src: make this work more than once in a row

We used to skip frame rate setup if the camera was already setup
with the requested frame rate. This breaks some cameras though,
causing them to not output data (several models of Thinkpad cameras
have this problem at least).
So, don't skip.

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

aacparse: only require two frames in a row when we do not have sync

This avoids a single bit error dropping two frames unnecessarily.
The two consecutive frames check is still required when we don't
have sync.

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

pulsesink: Trivial indentation fix

flacdec: Correct sample number rounding resulting in timestamp jitter

flacdec converts the src timestamp to a sample number, uses that internally, then reconverts the sample number to a timestamp for the output buffer.  Unfortunately, sample numbers can't be represented in an integer number of nanoseconds, and the conversion process was truncating rather than rounding, resulting in sample numbers and output timestamps that were often off by a full sample.

This corrects the time->sample convesion

breakmydata: element is not passthrough

multifilesrc: quiet debugging

deinterlace: change field handling through methods

This likely breaks stuff.  The good: all of the methods now create
field images aligned with input frames, without timestamp mangling.
The bad: this touches a lot of code, much of which is hairy and in
need of cleanup.  However, at this point we can reasonably create a
PSNR-based test.

multifilesink: reset ->streamheaders to NULL on _stop

Fixes invalid memory access reusing multifilesink

pulsesink: Allow writes in bigger chunks

There's no use in splitting the incoming data down to the segsize
limit - by writing as much as possible in one chunk, we increase
performance and avoid PulseAudio unnecessary rewinds.

Signed-off-by: David Henningsson <david.henningsson@canonical.com>

matroskademux: ensure no-more-pads is always emitted

In particular, do so even if failing to read while prerolling,
such as when reading from a partial file (eg, while it is being
downloaded).

This fixes a wedge in playbin2.

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

flacdec: avoid timestamp/offset tracking going out of sync

The libFLAC API is callback based, and we must only call it to
output data when we know we have enough input data. For this
reason, a single processing step is done when receiving a buffer.
However, if there were metadata buffers still pending, a step
intended for the first audio frame might end up writing that
leftover metadata. Since a single step is done per buffer, this
will cause every buffer to be written one step late.

This would add some latency (a bufferfull's worth), possibly
lose a buffer when seeking or the like, and also cause timestamp
and offset to be applied to the wrong buffer, as updates to
the "current" segment last_stop (from incoming buffer timestamp)
will be applied to an output buffer originating from the previous
incoming buffer.

This fixes the issue by ensuring that, upon receiving the first
audio frame, processing is done till all metadata is processed,
so the next "single step" done will be for the audio frame. After
this, we should keep to 1 input buffer -> 1 output buffer and so
avoid getting out of sync.

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

flacdec: bail on reserved value

Now that we look at the right bits, we can test against the reserved
value as we do for other fields.

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

flacdec: fix bit twiddling

Right shifting a 8 bit value by 8 bits is twice too much
to get the high 4 bits.

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

flacdec: warn if we see a variable block size where unsupported

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

spectrum: avoid crashing by resetting the correct number of channels

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

flacparse: fix off by one in frame size check

Yes, I was tracking another bug and the small test file I generated
to test with improbably just happened to trigger this, with a second
and last frame of 1615 bytes.

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

id3demux: remove specs from git as well now that parsing code is in -base

id3demux: use -base provided id3 tag parsing

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

jackaudiosrc: fix error message code

And also post 'not found' error if jackd is not even installed.

qtdemux: initialize bitrate variable and reset for each loop

Don't check eventually unset variable and don't accidentially use values from last
cycle.

rtspsrc: Properly error out if SDP contains no streams

Also fixes unitialized variable error on macosx.

ximagesrc: clear flags on buffer reuse

This will ensure a logically new buffer does not keep flags from
a previous use of that buffer (eg, DISCONT would be set on the first
buffer, and mistakenly kept when reused).

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

v4l2: take care not to change the current format where appropriate

Some drivers are buggy are will change the current format when
processing VIDIOC_TRY_FMT. Save and restore the current format
to ensure the format is kept unchanged.

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

v4l2src: Use fraction compare util function.

Use the fraction compare utility to compare function, not the
handcrafted one. The handcrafted one is buggy as it doesn't take into
account rounding error. For example comparing a framerate of 20/1 on a
camera configured as 30/1 fps would yield true: 1 == (1 * 20)/30 and not
re-configure the camera. Fixes #656104

matroska: Register new debug category

Register the matroskareadcommon debug category when the
plugin is loaded to avoid assertion output when debug is turned on.

qtdemux: soften assertion check on stream size

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

rtpjpegpay: Add support for H.264 payload in MJPEG container

See http://www.quickcamteam.net/uvc-h264/USB_Video_Payload_H.264_0.87.pdf

Fixes bug #655530.

jackaudiosink: Don't call g_alloca() in process_cb

g_alloca() is not RT-safe, so instead we should allocate the
memory needed in advance. Fixes #655866

docs: fix two more Since: tags

deinterlace: Fix Since tags for fieldanalysis related new properties

commit c1b100cf9c is after 0.10.29 and 0.10.30 was a branched release.
So fix Since tags from 0.10.29 to 0.10.31 for the new properties.

pulsesink: fix variable-set-but-not-used compiler warning with older pulse versions

rtpsession: properly init rtcp_min_interval

pulsesink: Add support for compressed formats

This adds support for various compressed formats (AC3, E-AC3, DTS and
MP3) payloaded in IEC 61937 format (used for transmission over S/PDIF,
HDMI and Bluetooth).

The acceptcaps() function allows bins to probe for what formats the sink
being connected to support. This only works after the element is set to
at least READY.

If the underlying sink changes and the format we are streaming is not
available, we emit a message that will allow upstream elements/bins to
block and renegotiate a new format.

pulsesink: Use the extended stream API if available

This uses the new extended API for creating streams. This will allow us
to support compressed formats natively in pulsesink as well.

pulsesrc: Add a source-output-index property

This exposes the source output index of the record stream that we open
so that clients can use this with the introspection if they want (to
move the stream, for example).

rtpssrcdemux: keep a ref on the src pad while using it

Prevent a possible race if clear_ssrc() is called between getting the pad and
doing the push.

Based on patch by <olivier.crete@collabora.com>

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

rtpssrcdemux: Make the pads lock recursive and hold it across the signal emit

We need to keep the lock held because we don't want a push before the "new-ssrc-pad"
handler has completed. But we may want to push an event from inside that handler, hence
the recursive mutex.

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

rtpssrcdemux: Use PADs lock

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

rtph264depay: Cope with FU-A E bit not being set

Some h264 payloaders are unfortunately buggy and don't correctly set the
E bit in FU-A NAL when they have ended. Work around this by assuming
such a fragmentation unit has ended when there was no packet loss and a
new NAL is started

ac3parse: Support switching alignment on-the-fly

This allows switching of alignment for E-AC3 streams at run-time. This
is requested by downstream elements via a custom event.

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

ac3parse: Add support for IEC 61937 alignment

When pushing out buffers over S/PDIF or HDMI, IEC 61937 payloading
requires each buffer to contain 6 blocks from each substream. This adds
code to collect all the frames needed to meet this requirement before
pushing out a buffer.

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

rtpsession: Always send application requested feedback in immediate mode

Send as many application requested feedback messages in immediate mode, even if they
have already been sent.

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

rtpsession: Don't let the computed RTP bandwidth fall too low

If it falls too low, the computed RTCP bandwidth will be near zero and
the RTCP thread will be stopped.

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

rtpsession: Wait longer to timeout SSRC collision

Using the current RTCP interval to timeout SSRC collision can lead to
collisions being timed out immediately if a BYE packet is sent because
it is sent immediately, so the interval is 0. This is not what we
want. So just set a static 10 times the default RTCP interval, it
should be enough

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

rtspsrc: set SOURCE flag at init time

Fixes #654816.

rtph264depay: Complete merged AU on marker bit

The marker bit on a RTP packet means the AU has been completed, so push it out
immediately to reduce the latency.

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

rtph264pay: Only set the marker bit on the last NALU of a multi-NALU access unit

An access unit could contain multiple NAL units, in that case, only the last
RTP packet of the last NALU should have its marker bit set.

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

multipart: fix compiler warning

auparse: avoid hanging on invalid short input

... as in such case there is no srcpad yet on which to forward EOS.

pulsesrc: Fix default value leaking

Remember to free the default value of client name, avoiding a
leak

rtph264depay: reset upon FLUSH_STOP

... which is particularly needed when merging NAL units, where not resetting
would lead to output of an older (pre-flush) AU (with unintended timestamp).

multifilesink: do not use g_slist_free_full

... as that is only in GLib 2.28, which is not yet required at this time.

multifilesink: add max-files property

Add max-files property to limit the number of files saved on disk.

API: multifilesink::max-files

multifilesink: refactor file opening and closing code

matroskademux: fix pixel-aspect-ratio if header has only one display variable

Current matroska demux calculates the pixel aspect ratio only if both
DisplayHeight and DisplayWidth are set, but it is legal to use only
one variable if the other is equal to PixelWidth or PixelHeight, at
least the mkclean utility is doing that. So this makse mkcleaned
files play correctly.

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

goom: fix build on PPC on openbsd

A missing sys/param.h include results in:

/usr/include/sys/proc.h:64: error: 'MAXLOGNAME' undeclared here (not in a
function)
/usr/include/sys/proc.h:285: error: 'MAXCOMLEN' undeclared here (not in a
function)

when compiling goom on openbsd/ppc. We can just remove the two sys/ includes
here, they are not needed for anything.

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

rtppcmApay/depay: Static clock rates on static payloads, dynamic on dynamic

Partially reverts 397dc60b

rtph264pay: Implement getcaps

Convert profile-level-id from RTP caps into video/x-h264 style caps (with profile and level)

rtspsrc: fix seeking regression

... introduced when shuffling around code for the async implementation
by setting state of source (and udp sources) in _play before downstream
flushing is undone.

ac3parse: fix buffer duration on blocks-per-frame change

The gst_base_parse_set_frame_rate call was predicated on a change to
sample rate, duration or profile. However, the block count per frame can
also change between packets, which would result in incorrect buffer
durations.

multifilesrc: Improve looping

Add start-index and stop-index properties.

multifile: add loop property to multifilesrc

Fixes: #652727

Signed-off-by: Jonny Lamb <jonnylamb@jonnylamb.com>
Signed-off-by: David Schleef <ds@schleef.org>

directsoundsink: 16-bit audio is signed, 8-bit is unsigned.

Pretending to handle 8-bit signed causes distorted audio when
actually given such audio, which you will get if passing 8-bit
unsigned through audioconvert ! audioresample, as audioresample
only handles 8-bit signed.  Fixes #605834.

Signed-off-by: David Schleef <ds@schleef.org>

matroskademux: handle blocks with duration=0

Some video frames, for example alt-ref frame in VP8, will be
never displayed. This is why it has duration=0.

This patch allow to use this duration.

Bug: 654175
Signed-off-by: Alexey Fisher <bug-track@fisher-privat.net>

qtmux: Add direct dirac mapping

pulsesink: prevent race condition causing ref leak

Since commit 8bfd80, gst_pulseringbuffer_stop doesn't wait for the
deferred call to be run before returning. This causes a race when
READY->NULL is executed shortly after, which stops the mainloop. This
leaks the element reference which is passed as userdata for the callback
(introduced in commit 7cf996, bug #614765).

The correct fix is to wait in READY->NULL for all outstanding calls to
be fired (since libpulse doesn't provide a DestroyNotify for the
userdata). We get rid of the reference passing from 7cf996 altogether,
since finalization from the callback would anyways lead to a deadlock.

Re-fixes bug #614765.

pulsesink: small cleanup of copy-paste code

pulsesink: remove unused member variable and misleading log message

Wim changed it in commit 8bfd80 so that pa_defer_ran is not read
anywhere.

The log message used to annotate a mainloop_wait call which is gone.

goom: Don't answer lantency queries before negotiation

jpegdec: avoid crashing on invalid input without components

flvmux: pass along segment info to collectpads

... so it can track this and be subsequently used to determine running time etc.

flvdemux: indicate raw format in aac caps

pulse: Increase ranks to PRIMARY + 10

So that pulsesrc/pulsesink get chosen over other possible PRIMARY
src/sinks by autoaudiosink.  Presumably, if pulse is available, it
is always preferred over another src/sink.

Fixes: #647540.

multipartmux: Add \r\n to tail of pushed buffers

Clients such as Firefox require the \r\n after the payload.

matroskademux: avoid looping when searching for clusters

Fixes some bugs that results in the demuxer looping when seaching
for clusters in non-finalized files.

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

matroskaparse: fix reference counting of parse->streamheader

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

Signed-off-by: David Schleef <ds@schleef.org>

jpegenc: Don't round up size of encoded buffers

For some reason, in code dating to 2001, encoded jpeg buffers were
rounded up to multiples of 4 bytes.  With the added bonus that the
extra bytes are unwritten, causing valgrind issues.  Oops.  I can't
think of any reason why JPEG buffers need to be multiples of 4 bytes,
so I removed the padding.  There might be some code somewhere that
depends on this behavior, so if this needs to be reverted, please fix
the valgrind issues.

qtmux: free date tag

aacparse: not so greedy minimum frame size

Fixes #653559.

configure: remove non-pkg-config check for shout

Fixes: 653327