wavparse: Store size of data tag in a 64 bit integer locally too

Otherwise we will clip the DS64 value of RF64 files to 32 bits again.

wavparse: Use 64 bit scaling functions now that fact is a 64 bit integer

wavparse: support rf64 format

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

multipartdemux: Ensure caps before pad added.

This stores the stream-start, sets caps, and then adds the pad,
which ensures that the caps are set for the "pad-added" callback.

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

flvmux: Fallback to PTS if DTS is missing

Fixing a regression introduce when fixing:
https://bugzilla.gnome.org/show_bug.cgi?id=731352

imagefreeze: Remove impossible error condition

We return EOS after the first buffer, and GstPad will make sure now that we
won't get any other buffer afterwards until a flush happens. No need to check
for it ourselves.

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

gdkpixbufdec: EOS and NOT_LINKED are no errors in general

Don't post an error message for them but let upstream handle
anything accordingly.

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

flvmux: Correctly offset timestamp

The previous method would break AV sync in the case audio or video
didn't start at the same point in running time.

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

flvmux: Save dts from buffer

We no longer set dts in muxed buffer. This would lead to encoding tags
with timestamp 0 instead of the timestamp of previous buffer.

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

flvmux: Ensure Timestamp starts at 0

FLV documentation stipulates that timestamp must start at zero.
In order to respect this rule, keep the first timestamp around
and offset the timestamp from this value. This allow for longer
recording time in presence of timestamp that does not start
at 0 already.

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

flv: Tag timestamp are DTS not PTS

The tags in FLV are DTS. In audio cases, and for many video format this makes
no difference, but for AVC with B-Frames, PTS need to be computed from
composition timestamp CTS, with PTS = DTS + CTS.

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

jitterbuffer: Allow rtp caps without clock-rate

The jitterbuffer shouldn't force clock-rate on its sink pad, this will cause a negotiation issue since rtpssrcdemux doesn't have the clock-rate and doesn't add it to the caps. The documentation states that the clock-rate can either be specified through the caps or through the request-pt-map signal, so we must remove clock-rate from the pad templates and we must accept the GST_EVENT_CAPS if the caps don't have the clock-rate.

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

qtdemux: avoid crashing on dash streams

DASH/fragmented moov might have no samples as those are carried
in moof fragments. Avoid crashing or failing the stream because
of that.

examples: use 'post-messages' property instead of deprecated 'message' property

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

udp: fix udpsrc documentation

udpsrc gtk-doc documentation refers to sockfd and closefd properties which has
been removed. This patch replaces those references to socket and close-socket
respectively.

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

qtmux: Make the default timescale 1/1800 second

The old default timescale of 1 millisecond produces irrational
numbers for a lot of framerate/audio-packet-duration multiples.
1/1800 is a nicer number, as it tends to produce better fractions
and therefore slightly higher accuracy overall

matroska: Use gst_video_guess_framerate() function

Remove local framerate guessing function in favour of
the new gst_video_guess_framerate() function.

qtdemux: Improve framerate calculation/guessing

Change the way the output framerate is calculated
to ignore the first sample (which is sometimes truncated
in my testing) and use the new gst_video_guess_framerate()
function to recognise common standard framerates better.

Remove the code that was sorting the first 20 sample
durations and then ignoring the result.

videomixer: Use the best width/height/etc if downstream can handle that

Before it was always using whatever downstream preferred, while
the code and documentation claimed something different.

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

videomixer: Avoid double free of VideoConvert

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

flvdemux: fix indentation

flvdemux: un-break duration querying

Commit 2b9493b5 broke this in two ways: a) we should only
pass duration queries in TIME format upstream (or at least
not those in DEFAULT or BYTE format), and b) we mustn't
overwrite the default value of 'res' from TRUE to FALSE
and not set it again later. This led to bogus durations
being reported for FLV playback from file, because TIME
queries would fail (as 'res' had been set to FALSE) and
parsers then do a BYTE query as fallback and try to
guesstimate something in return, which of course goes
horribly wrong since the BYTE size returned is for the
muxed file.

videobalance: Allow any raw caps in passthrough mode, not just the ones we handle

videobalance: Allow ANY capsfeatures, but only in passthrough mode

When changing the properties to not be in passthrough mode anymore,
we will only accept caps we can process ourselves, potentially causing
a not-negotiated error.

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

docs: update for git

configure: build ximagesrc again when checks succeed

Third time lucky, hopefully.

configure: fix x11 checks to be non-fatal again

Must pass an action-if-not-found argument to
PKG_CHECK_MODULES or it will error out when
it can't find the module requested. Also fix
AC_CHECK_LIB usage, extra libs argument was
in the wrong place.

qtdemux: forward DISCONT from upstream to the output streams

This makes sense in DASH reverse playback, where the upstream dashdemux
will download DASH segments in reverse order, but push their buffers
forward to qtdemux and mark each segment start as DISCONT. This needs
to be forwarded downstream to the parser/decoder, otherwise it won't work.

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

configure: use pkg-config to detect x11 and simplify checks

AC_PATH_XTRA macro unnecessarily pulls in libSM and libICE.

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

tests: rtp-payloading: adjust test data to avoid NAL chopping

... and correspondingly unexpected buffer sizes.

speexenc: Improve annotation of internal function

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

shapewipe: Unref caps and element after usage

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

qtdemux: improve debug logging of fourccs

If we can't show ASCII, at least show them
in big endian order.

qtdemux: add support for 'wma ' mapping as found in some ismv files

e.g. To_The_Limit_720_2962.ismv

qtdemux: add support for 'vc-1' mapping as found in some ismv files

e.g. To_The_Limit_720_2962.ismv

rtph263ppay: Unref pad template caps after use

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

videomixer: Unref allowed caps after usage

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

imagefreeze: Unref pad template caps after usage

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

navseek: Unref peer pad after usage

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

rtpmux: Unref pad template caps after usage

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

rtph264pay: append packetization mode parameter to SDP

Append packetization-mode parameter to SDP description.
Packetization mode signals the properties of an RTP payload type.

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

isomp4/qtmux: Write correct file duration when gaps exist.

When writing out a trak with an edit list, make sure the
overall file duration is also updated to reflect the
lengthening of the stream.

Add some more debug to qtdemux to warn about streams that
are longer than the file and get truncated.

rtspsrc: Push the correct segment in TCP mode when seeking

rtph264pay: unbreak au aligned byte-stream payloading

rtph264pay: append profile-level-id to SDP

Append profile-level-id to SDP if available.

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

Makefile: Add usage of build-checks step

Allows building checks without running them

ximagesrc: Fix warning about missing return value

ximagesrc: Add missing return value to Buffer dispose function

Depending ont he build, the method could return FALSE, hence never
free the buffers, or already TRUE and lead to a crash:

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

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