interleave: set output caps layout to interleaved

Set output caps layout independently from input caps layout which can
be either non-interleaved or interleaved.

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

v4l2bufferpool: clear gcond

Revert "v4l2bufferpool: Workaround elements not requesting any buffers"

This was a tempory workaround, we should fix the encoders that do not
negotatiate the amount of buffers they need.

This reverts commit d03bcba3db15d06dbdea6b776a6f28ed2f03272a.

v4l2object: Don't share own pool if min exceed V4L2 capacity

If the minimum required buffer exceed V4L2 capacity, don't share down
pool. This allow support very high latency, like with x264enc default
encoding settings.

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

v4l2object: query minimum required buffers for output

Some v4l2 devices could require a minimum buffers different from default
values. Rather than blindly propose a pool with min-buffers set to the
default value, it ask the device using control ioctl.

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

v4l2sink: use directly 'obj' instead of 'v4l2sink->v4l2object'

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

v4l2: set debug messages according to device type and IO mode

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

v4l2object: Remove is_active checks

These checks are no longer required with recent change to the bufferpool. This
should allow changing the configuartion, hence the way forward renegotiation
support.

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

qtdemux: fix language code parsing for 3-letter codes starting with 'a'

And handle special value for 'unspecified' explicitly.

https://developer.apple.com/library/mac/documentation/QuickTime/QTFF/QTFFChap4/qtff4.html

jpegenc: Add support for encoding from NV21 and NV12

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

Back to development

Release 1.4.0

Update .po files

po: Update translations

videobox: Don't overwrite the first component with the alpha value for BGRx

Instead leave the x component unset when filling the borders.

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

aacparse: Properly report in the CAPS query that we can convert ADTS<->RAW

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

rgvolume: Avoid taking unnecessary refs

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

rtpdtmfmux: Avoid taking an unnecessary ref

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

po: update POTFILES

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

v4l2bufferpool: Fix copy threshold implementation

We cannot allocate new buffer in acquire, otherwise the base class
is not aware and get confused. Instead, copy in _process(). This leads
to crash on finalize.

Fixes regression, see https://bugzilla.gnome.org/show_bug.cgi?id=732912

Release 1.3.91

Update .po files

v4l2allocator: Use qdata instead of parenting to DmabufMemory

Parenting V4l2Memory to DmabufMemory was in conflict with recent
optimization in DmabufMemory to avoid dup(), and didn't work with
memory sharing. Instead, use a qdata and it's destroy notify.

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

po: Update translations

v4l2bufferpool: Workaround elements not requesting any buffers

This is a workaround for element that don't request buffers when
they should.

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

tests: fix pipeline leak in videocrop test

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

examples: client-rtpaux: Release reference to parent when done

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

rtspsrc: fix query leak

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

wavenc: Return not-negotiated if we got no caps or caps negotiation failed

And do it always, not inside a g_return_val_if_fail().

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

v4l2src: Ensure internal pool activation

Before we would hit an assertion "'gst_buffer_pool_is_active (bpool)' failed"
if the internal pool was not used to push buffer downstrea, hence not
given to the baseclass.

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

videomixer: fix double unlock in segment seek segment code path

We only want to unlock if we push an event downstream and
jump to done_unlock label afterwards. We would also unlock
in case of a segment seek and then unlock again later, and
nothing good can come of that.

(This code looks a bit dodgy anyway though, shouldn't it
also bail out with FLOW_EOS here in case of a segment seek
scenario, just without the event?)

tests: qtmux: suppress glib criticals caused by testing deprecated dts methods

avidemux, wavparse: Print invalid fourcc in hex

Previously this was printed as characters which caused later processing
of the error message to sometimes warn about non-UTF-8 characters.

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

v4l2object: Pool might be NULL in decide allocation

If special stride is needed and downstream don't support VideoMeta,
pool might be NULL in order to let the baseclass create a generic
pool­. This would lead to assertion with on Exynos with:

gst-launch-1.0 -v filesrc location=mov ! qtdemux ! h264parse ! \
                    v4l2video8dec ! fakesink

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

v4l2bufferpool: Handle FD error during poll

This will ensure we fail earlier if something unrecoverable
happens.

v4l2bufferpool: Wait before polling if queue is empty

In kernel before 3.17, polling during queue underrun would unblock right
away and trigger POLLERR. As we are not handling POLLERR, we would endup
blocking in DQBUF call, which won't be unblocked correctly when going
to NULL state. A deadlock at start caused by locking error in libv4l2 was
also seen before this patch. Instead, we wait until the queue is no longer
empty before polling.

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

rtspsrc: fix for mikey api change

v4l2: fix probing and enumeration of stepwise frame sizes

The code enumerating STEPWISE framesizes would start from
(min_w, min_h) and then add (step_w, step_h) to get the
next framesize. However, it should really allow any width
from min_w to max_w with step_w and same for heights.
Secondly, we would add and probe each individual stepped
frame size to the caps as separate structure, which would
lead to hundreds if not thousands of structs ending up in
the probed caps. Use integer ranges with steps instead.

This was particularly noticable with the Raspberry Pi Cam.

https://bugzilla.gnome.org/show_bug.cgi?id=724521
https://bugzilla.gnome.org/show_bug.cgi?id=732458
https://bugzilla.gnome.org/show_bug.cgi?id=726521

v4l2object: drop workaround for misbehaving TRY_FMT

This workaround from 2011 was causing 25 S_FMT ioctls to be sent
to my UVC webcam from under gst_v4l2_object_get_caps as it probes
all the formats. In total, this adds up to about 5 seconds of
execution time, or a 10 second delay while starting up cheese.

These ioctls come from a workaround from 2011 where TRY_FMT might
make changes to hardware settings, so S_FMT was used to restore
the original config:
https://bugzilla.gnome.org/show_bug.cgi?id=649067

The driver bug is now assumed fixed. Remove the workaround to fix the
long startup delay.

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

videomixer: reset QoS on segment event

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

matroskademux: send gap events instead of segment tricks

This fixes missing frames from being time skipped.

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

rtpsession: Fix memory leaks in unit test

rtpbin: Don't leak caps

pulsesrc: Fix compiler warning when compiling with G_DISABLE_ASSERT

rtpssrcdemux: Fix compiler warning when compiling with G_DISABLE_ASSERT

matroskamux: Fix compiler warnings when compiling with G_DISABLE_ASSERT

deinterlace: Fix compiler warnings when compiling with G_DISABLE_ASSERT

pulse: fix compiler warnings when compiling with -DG_DISABLE_ASSERT

Compiler complains about uninitialised variables in the impossible
'default' code path in device provider source/sink switch-case.

v4l2: fix compiler warnings when compiling with -DG_DISABLE_ASSERT

Compiler complains about uninitialised variables in the impossible
'default' code path in device provider source/sink switch-case.

tests: matroskaparse: fail on errors and disable pull mode test

Actually look for error messages on the bus and fail if there
is one before the EOS message. Disable pull mode test which is
pointless as long as matroskaparse only supports push mode
(pull mode support has not been ported over to 1.0).

matroskaparse: don't error out if there's not enough data in the adapter

gst_matroska_parse_take() would return FLOW_ERROR instead of
FLOW_EOS in case there's less data in the adapter than requested,
because buffer is NULL in that case which triggers the error
code path. This made the unit test fail (occasionally at least,
because of a bug in the unit test there's a race and it would
happen only sporadically).

videomixer: Update dist generated ORC files

videomixer: Update videoconvert code from -base

And also rename the remaining symbols to prevent conflicts
during static linking.

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

autovideosrc: use videotestsrc as fallback element instead of fakesrc

fakesrc doesn't announce video caps, so most video pipelines will
just error out with not-negotiated if a fallback element is created.

autoaudiosrc: use audiotestsrc as fallback element instead of fakesrc

fakesrc doesn't announce audio caps, so most audio pipelines will
just error out with not-negotiated if a fallback element is created.

Release 1.3.90

Update .po files

Rename GstDeviceMonitor to GstDeviceProvider

videobox: Add unit test

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

videomixer: Declare as Compositor in 'klass'

flvdemux: fix speex caps

Decoder complains about "notification: Invalid mode encountered.
The stream is corrupted" though, even if it works, so there's
probably something wrong with the generated codec headers.

flvmux: fix speex in FLV

Speex in FLV is always mono @ 16kHz, see
http://download.macromedia.com/f4v/video_file_format_spec_v10_1.pdf
section E.4.2.1: "If the SoundFormat indicates Speex, the audio is
compressed mono sampled at 16 kHz, the SoundRate shall be 0, the
SoundSize shall be 1, and the SoundType shall be 0"

Also see https://bugzilla.gnome.org/show_bug.cgi?id=683622

isomp4: Add object type id and fourcc for DTS/DTS-HD

Enables playback for files with DTS audio tracks.
Also add an extra AC-3 variant fourcc from Nero

videomixer2: Solve segmentation fault when src caps are configured

Change function pointers to NULL while holding the lock to avoid
race conditions

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

jitterbuffer: improve SR packet handling

Implement 3 different cases for handling the SR:

 1) we don't have enough timing information to handle the SR packet and
    we need to wait a little for more RTP packets. In that case we keep
    the SR packet around and retry when we get an RTP packet in the
    chain function.

 2) the SR packet has a too old timestamp and should be discarded. It is
    labeled invalid and the last_sr is cleared.

 3) the SR packet is ok and there is enough timing information, proceed
    with processing the SR packet.

Before this patch, case 2) and 1) were handled in the same way,
resulting that SR packets with too old timestamps were checked over and
over again for each RTP packet.

tests: add udpsink test to check client add/remove

tests: port udpsink tests to 1.0

They all seem a bit pointless though.

avimux: Add UYVY format

gstrtpssrcdemux: manage ssrc of RTCP RR packets

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

wavparse: Update offset after parsing adtl chunk

Otherwise we will parse it over and over again without ever
getting past it.

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

osxvideosink: remove legacy code for passing a window handle

"have-ns-view" and the "embed" property was kept in 0.10 for
backwards compatibility but it's no longer used in favor of
the GstVideoOverlay interface

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

Back to development

matroskademux: Don't call GST_DEBUG_OBJECT() and other macros with non-GObject objects

It will crash with latest GLib GIT and was never supposed to work before
either.

Release 1.3.3

Update .po files

po: Update translations

pulse, v4l2: update for device "klass" -> "device-class" rename

multiudpsink: optimisation: avoid unnecessary memory ref/unrefs

We know the buffer will stay valid and we will also not
modify the buffer, we just want to send out the data.

multiudpsink: avoid some unnecessary run-time type checks

rtspsrc: pass the stream id when asking for crypto params

This way the app can choose different parameters for each stream.

rtspsrc: add support for key length parameters

This patch adds supports for the incoming key management parameters for
encryption and authentication key lengths.

It also adds a new signal request-rtcp-key that allows the user to
provide the crypto parameters and key for the RTCP stream.

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

vp8depay: fix header size checking

Use a different variable name to make it clear that we are calculating
the header size.
Correctly check that we have enough bytes to read the header bits. We
were checking if there were 5 bytes available in the header while we
only needed 3, causing the packet to be discarded as too small.

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

rtph264pay: propagate the GST_BUFFER_FLAG_DISCONT flag

Similarly to what we did with the DELTA_UNIT flag, this patch
propagates the DISCONT flag to the first RTP packet being used to transfer a
DISCONT buffer.

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

rtph264pay: propagate the GST_BUFFER_FLAG_DELTA_UNIT flag

Downstream elements may be interested knowing if a RTP packet is the start
of a key frame (to implement a RTP extension as defined in the
ONVIF Streaming Spec for example).

We do this by checking the GST_BUFFER_FLAG_DELTA_UNIT flag we receive from
upstream and propagate it to the *first* RTP packet outputted to transfer this
buffer.

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

gstrtpmp4gpay: propagate the GST_BUFFER_FLAG_DISCONT flag

Propagate the DISCONT flag to the first RTP packet being used to transfer
a DISCONT buffer.

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

rtpjpegpay: propagate the GST_BUFFER_FLAG_DISCONT flag

Propagate the DISCONT flag to the first RTP packet being used to transfer
a DISCONT buffer.

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

avidemux: don't leak flow combiner

rtpjp2kpay: pre-allocate buffer-list of the right size

rtpjpegpay: pre-allocate buffer list of the right size

rtpmp4vpay: pre-allocate buffer list of the right size

rtpvp8pay: allocate bitreader on the stack

rtpvp8pay: post error message on bus on error and don't use g_message()

rtpvp8pay: couple of minor optimisations

Pre-allocate buffer list of the right size to avoid re-allocs.
Avoid plenty of double runtime cast checks and re-doing the
same calculation over and over again in rtp_vp8_calc_payload_len().
Only call gst_buffer_get_size() once.

rtpgstpay: pre-allocate buffer list of the right size

To avoid re-allocs.

rtph264pay: pre-allocate bufferlist of the right size

To avoid unnecessary re-allocs.

rtph264pay: push single buffer directly, no need to wrap it in a bufferlist

No point in a buffer list if we just have one single
buffer to push. Fix up unit test to handle that case
as well.

rtpvrawpay: make chunks per frame configurable

Bit of a misnomer because it's really chunks per field
and not per frame, but we're going to ignore that for
the time being.

rtpvrawpay: remove unused variables

rtpvrawpay: pre-allocate buffer lists of sufficient size

Avoids unnecessary reallocs when appending buffers
to the bufferlist.

rtpvrawpay: micro-optimise variable access in inner loop

Store some values that don't change during the execution
of the inner loops locally, so the compiler knows that too.