Move m_pprevFrame free just before the clone of the latest frame for readability

camerabin2: Make viewfinder's queue a thread limit only

Reduce the viewfinder queue limits to only allow it to store
one buffer, preventing the queue from holding old buffers for
too long. This also avoids showing slightly outdated frames on
the viewfinder when the source has already produced new ones
and improves the buffer recycling rate, important for sources
that use bufferpools.

basevideoencoder: fix element leak

and this concludes an hour of yelling at the bloody test failing,
only to track down the problem not being in the test.

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

tsdemux: In push mode, do not start pull loop if upstream seek fails

hlsdemux: do not make the update_thread joinable

hlsdemux: Ensure the task is paused in case of cache error

hlsdemux: stop the threads in the proper order

first pause the task, then stop all fetchers, then stop the update thread
then pause the task again, since it might have been restarted by
another thread in the meantime

hlsdemux: do not post an error if we seek/cancel during caching of fragments

hlsdemux: if we're canceling a fetcher, we must stop its thread

hlsdemux: lock the update_thread mutex before stopping it

hlsdemux: lock the fetcher mutex before stopping it

hlsdemux: do not allow two fetchers to run simultanously

hlsdemux: Stop the streaming task on PAUSED_TO_READY state change

tests: make camerabin2 test build

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

dtmf: Add more debug

dtmf: Max event type is 15

dtmfsrc: Align DTMF sound buffers with last-stop from event

Also make sure the timestamps never go backwards

rtpdtmfsrc: Correctly recognize the end of a buffer

rtpdtmfsrc: Make sure rtpdtmfsrc timestamps don't overlap

rtpdtmfsrc: Put the inter digit interval at the end, not at the start

The reason is to let rtpdtmfmux drop buffers during the inter digit interval,
this way, there will be more silence around the DTMF tones so IVFs will have
a better chance recognizing them.

rtpdtmfsrc: Start at the last_stop from the start event if there was one

The goal is to try to not have a GAP between the audio and the DTMF

rtpdtmfmux: Add last-stop to dtmf-event upstream events

Add the running time of the last outputted buffer to the
upstream "dtmf-event" events so that the dtmf source does not
leave a gap.

rtpdtmfsrc: Respect ptime from the caps

Respect the ptime from the caps for the DTMF packets

rtpdtmfsrc: Just error out if there is no clock

kate: make gst_kate_util_is_utf8_string() inline to fix compiler warning with older libkate versions

gstkateutil.c:230:1: error: 'gst_kate_util_is_utf8_string' defined but not used

rsndvd: ensure metadata is writable when writing to it

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

tsdemux: Use FALSE instead of 0 for bool parameter

tsdemux: Need to send a newsegment after a seek

hlsdemux: Send buffering messages on the bus

hlsdemux: cache the number of fragments we are required to cache

tsdemux: Forward seek/duration query/event to the sinkpad

camerabin2: Optimize start up a little by removing some pad checks

Some pad checks are guaranteed to work, so we can disable them
by using gst_element_link_pads_full

camerabin2: Remove from experimental

Remove camerabin2 from experimental, allowing it to be built
without needing to pass --enable-experimental on configure.

camerabin2: Always set encodebin flags when going to ready

When going from NULL to READY we want the encodebin flags to be
set, and not only after creating encodebin (on the first run)

camerabin2: Add new entry to flags property

Adds a new entry to camerabin2 flags to disable viewfinder
conversion elements.

camerabin2: viewfinderbin: Add property for disabling converters

Adds a new property to disable converters elements for performance
reasons. It should only be used if the application knows they aren't
needed.

tsdemux: do not try to compare a PCR with itself for statistics

Especially as one of the logs will cause a divide by zero.

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

tsdemux: do not keep a dangling pointer

When removing the current program, it will get freed by the
hash table removal callback, so ensure we clear our pointer
to it.

Fixes a crash later on in gst_ts_demux_push trying to access it.

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

kate: pass on timed event metadata as tags when we can

Metadata attached to other objects that may be used in this
event are not passed, as is binary metadata.

configure: Add AC_MSG_CHECKING() to the VP8E_SET_TUNING presence check

vp8: probe for the new tuning API to keep building with older libvpx

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

rsvgdec: use input buffer timings if possible

SVG data may come through multiple buffers, so keep track of the
timestamp of the first buffer, and use it in preference.

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

kate: do not consider an empty SPU to be a fatal error

Instead, just encode nothing and go on.

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

kate: do not clear out high bits from display area

http://dvd.sourceforge.net/spu_notes does not mention that high bits
are to be masked, and not clearing them makes a sample work, where
clearing them yielded left > right.
History from the dvdspu plugin, from where this code was copied,
does not shed any light, as tracing this code's origin shows
the same bitmasks being there in 2007 when it was imported.

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

dvdspu: do not clear out high bits from display area

http://dvd.sourceforge.net/spu_notes does not mention that high bits
are to be masked, and not clearing them makes a sample work, where
clearing them yielded left > right.
History does not shed any light, as tracing this code's origin shows
the same bitmasks being there in 2007 when it was imported.

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

mpegtsbase: First try if upstream handles seeking in TIME format before seeking in the demuxer

tsdemux: First send SEEKING queries upstream

tsdemux: Send DURATION query upstream first

hlsdemux: Use FALSE instead of 0 for a bool parameter

hlsdemux: switch pads when we seek (need_segment) since tsdemux doesn't cope well with a reset

hlsdemux: Add a FIXME comment. loop task should be a pad task

hlsdemux: Do not seek in a live stream

hlsdemux: Do not take the pad stream lock, use the task's lock instead

hlsdemux: Implement EVENT_SEEK

hlsdemux: Implement the GST_QUERY_SEEKING properly

hlsdemux: Implement correctly the GST_QUERY_DURATION

hlsdemux: Do not use GST_TASK_WAIT in the task function

The task function uses GST_TASK_WAIT which does a g_cond_wait giving it
the GST_OBJECT_GET_LOCK of the task. The mutex gets locked when
g_cond_wait returns, so if we don't lock/unlock it, it will
stay locked forever, preventing the task from ever finishing.
We shouldn't lock the task object lock, so let's remove the GST_TASK_WAIT
and make the task pause instead if there are no buffers in the queue.

tsdemux: do not send a new-segment on stream removal

When a program is changed, stream_added is called which sets the
need_newsegment to TRUE, then stream_removed is called, which calls
the flush_pending_data, which checks for the newsegment and causes
it to send a new-segment.

We must not send the newsegment when flushing the pending data on the
removed stream. We should only push it when flushing data on the newly
added streams (after they finish parsing their PTS header)

tsdemux: Fix newsegment when switching programs

If a program/stream is changed, then a newsegment is sent which must
not be the same as the base segment since it happens later. We must
shift the start position by the time elapsed since the newsegment
and the current PTS of the stream

tsdemux: interpolate gap and fix timestamps depending on upstream segment

hlsdemux: Keep a separate variable for the current variant used

By using a separate variable, first it allows us to sort the lists
of alternates but keep the pointer on the first occurence in the main
playlist (to respect the spec of starting with the bitrate specified
first in the main playlist). It also avoid playing with the lists variable
which should be used to store the list of playlists and not as a pointer
to the current one.
Also fixes a memleak with the g_list_foreach freeing the lists, if it wasn't
pointing to the first element of the list.

decklink: fix compilation on Linux

decklink: Add subdevice property

decklink: add support for Windows

Builds with MSVC, not tested in MinGW or anything else. Added
COM handling code, borrowed from dshowvideosink. Moved Linux
specific files to ./linux and added Windows specific files to
./win (generated from IDL files in VS9). Added macros to handle
BSTR and defined bool as BOOL.  Fixes #655362.

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

decklink: improve frame handling

Send NEWSEGMENT on start. Send warning instead of error on
dropped frames.

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

decklink: fix debugging in capture.cpp by making category extern

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

decklink: use mutexes from glib instead of pthread

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

removesilence: new plugin

Fixes: #597822.

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

camerabin2: Prevent audiosrc from sending undesired eos

Basesrc derived classes send an eos when they change state
from paused to ready and that breaks video recordings on camerabin2
as it makes the whole audio branch pads flushing.

Prevent it by using a pad probe that only allows the eos to pass
when it is caused by a stop-capture action.

tests: camerabin2: Use audiotestsrc on tests

Use audiotestsrc on tests instead of using the system's default
audiosrc, avoiding any problems that a local audio setup might
have.

camerabin2: Use gst_pad_send_event instead of eventfunc macro

tests: camerabin2: Improving tests

Adding some additional checks for custom camerabin2 messages

Also improves reliability of the tests by waiting for the preview-buffer message.
This message is generated from the preview pipeline, and might arrive later than the
image/video-done from the main camerabin2 pipeline, the tests need to wait for it even
after getting the image/video-done.

camerabin2: Avoid assertions when plugin creation fails

Capsfilters are created on the constructor and their properties can
be set/get from camerabin2's set/get_property functions. The user with
a broken setup would cause assertions when trying to set/get the
capture caps of this camerabin2.

A proper missing-plugin message will be posted when the user tries to
set camerabin2 to READY state.

tsmux: remove unnecessary check

remove an if (best) check where best != NULL has already been checked

tsbase: reinitiate segment at flush-stop event only

hlsdemux: Set buffer timestamp since we send a TIME segment

hlsdemux: Allow bandwidth switching during caching of fragments

hlsdemux: set the pad's caps before adding it

ofa: fingerprint creation can fail

There does not seem to be a way to retrieve information
about why failure occured, unfortunately.
When failure occurs, do not post a NULL fingerprint tag.

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

ofa: cleanup

Avoid repeating casting to GstAudioFilter when one will do
Use object related logging to make it easier to get just the logs we want

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

ofa: do not create fingerprint on EOS if record is FALSE

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

vp8enc: Remove unused and useless variable in tags handling

basevideo: Fix locking, especially if both pads have different streaming threads

basevideo: Don't call g_type_class_peek_parent() in class_init

This is already done by the GObject boilerplate macro

aiffmux: drop data after 4ish GB and moan

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

aiffmux: use guint32 for guint32 parameters

This makes explicit that the range is limited.

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

aiffmux: avoid integer overflow

These values are 32 bits, and width is a multiple of 8.

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

celtdec: make this compile with git libcelt

celt_mode_info was removed, in favor of a _ctl.

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

ofa: don't crash when there's no data to create a fingerprint from

Fixes a crash when no data flowed through.

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

mpeg4videoparse: Use gst_bit_reader_skip to skip more than 32 bits.

GET_BITS is a macro for gst_bit_reader_get_bits_uint32, which cannot
read more than 32 bits and will fail in this case where it is called
to read 79 bits. Since we want to skip those bits, gst_bit_reader_skip
is more appropriate in this case.

qtkitvideosrc: get the framerate from caps

...and configure QTKit accordingly. Hardcoding to 30fps was pretty -bad.

tests: camerabin2: Add test for audio-filter property

Checks that the audio-filter is properly plugged in the pipeline
and receives buffers when recording a video.

camerabin2: Add audio-filter property

Adds a property to add a custom GstElement to the audio
branch of the pipeline. This allows the user to do custom audio
processing/analysis when recording videos.

basevisualizer: add more shader variants and simplify code

Use macros to simplyfy the shading code. Those will ease to add support for
other colorspaces in the future. Add more variants for the shading (left,right,
horiz-in, vert-out, vert-in).

basevisualizer: more docs

baseaudiovisualizer: protect config with a lock

Before it was easy to crash the elements when using a ximagesink and triggering
renegotiation by resizing.

spectrascope: make a copy of the audiodata before downmixing and windowing

The buffers come from the adapter and the data might overlap. We don't want to
modify it in place.

basevideoencoder: Proxy the width/height/framerate/PAR constraints of downstream caps to upstream

This allows to specify constraints on the compressed downstream caps
by muxers or capsfilters, which will then be forwarded to upstream
and allows video converters to fulfill the constraints.

Code based on Mark Nauwelaerts audio encoder base class.

basevideoencoder: Remove old ::getcaps() comment

diracenc: Update for basevideoencoder ::get_caps() removal