srtpdec: Fix comparison compiler warning

error: comparison of constant -1 with expression of type
'GstSrtpCipherType' is always false

motioncells: Remove unused private fields

segmentation: Fix integer underflow check

error: comparison of unsigned expression < 0 is
always false [-Werror,-Wtautological-compare]

dfbvideosink: Fix unitialized variable compiler warning

bz2: Fix typo in header include guards

vdpau: Fix comparison compiler warning

error: comparison of constant -1 with expression
of type 'const GstVideoFormat' is always false

uvch264src: Fix unitialized variable compiler warnings

id3tag: Fix uninitialized variable compiler warning

error: variable 'image_type' is used uninitialized whenever
'if' condition is false [-Werror,-Wsometimes-uninitialized]

dvdspu: Remove unused variable

mpegts: pmt: Detect descriptor length correct

mpegts: Correctly check for PAT,PMT,CAT,TSDT

mpegts: Fix some packetizing bugs

- Length of NIT stream descriptors was not detected correct
- Reserved bits was not set according to EN 300 468, ISO/IEC 13818-1
- Also set output data size if the section was previously packetized

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

mpegtspacketizer: Remove or move variable assignments

To where they're needed

dvb-descriptor: Remove unneeded increment

dvb-descriptor: Fix comparision

we want to stop if one OR the other is NULL.

mpegts: Updated docs with new API

mpegtsmux: Use mpeg-ts lib for PAT/PMT sections

Rewrite of the PAT/PMT section handling to use the mpeg-ts library

mpegtsmux: Add support for muxing SI tables

The muxer is now able to include DVB sections in the transport stream.

The si-interval property will determine how often the SI tables are
muxed into the stream.

The section is handled by the mpeg-ts library. Below is a small example
that will include a Netork Information Table with a Network Name
descriptor in the stream.

GstMpegTsNIT *nit;
GstMpegTsDescriptor *descriptor;
GstMpegTsSection *section;
GstElement *mpegtsmux;

gst_mpegts_initialize ();

nit = gst_mpegts_section_nit_new ();
nit->actual_network = TRUE;

descriptor = gst_mpegts_descriptor_from_dvb_network_name ("Network name");
g_ptr_array_add (nit->descriptors, descriptor);

section = gst_mpegts_section_from_nit (nit);

// mpegtsmux should be retrieved from the pipeline
gst_mpegts_section_send_event (section, mpegtsmux);
gst_mpegts_section_unref (section);

mpegts: Support registration and custom descriptor

Support for registration descriptor (0x05)
Add function to create a descriptor with custom tag and data

mpegts: Add creation of DVB Subtitling descriptor

Descriptor tag is 0x59

mpegts: Support parsing of DVB Teletext descriptor

Descriptor tag is 0x56

mpegts: Add support for creating PAT/PMT

mpegts: Add program_number to GstMpegTsPMT

mpegts: Add network name descriptor construction

Add function to create a Network Name DVB descriptor.

mpegts: Add support for creating a NIT section

Functions that will enable user to create Network Information Tables.

mpegts: Add network_id to GstMpegTsNIT

The network_id is stored in the subtable extension. Make a field
in the GstMpegTsNIT for better code readability

mpegts: Add functions to send sections as events

Added function that enables the user to send a GstMpegTsSection as
an event to a GstElement. (i.e. mpegtsmux)

mpegts: Add functions to packetize section

Sections needs to be packetized for use in i.e. mpegtsmux.
These functions handles the generic common parts of a GstMpegTsSection

mpegts: Add init functions for descriptor/section

mpegts: Add function to encode UTF8 strings

This can be used to create descriptors with appropriate character maps

mpegts: Use big endian UCS-2 coding

Parsing of control codes requires a big endian character map

mpegts: Add support for two-way iconv conversions

To use in conversions from UTF-8 to another character map

applemedia: Fix libtool usage

--tag=CC is needed for static build

mpegtsdemux: Update plugin license field

The original code (old mpegtsparse) from which this plugin was based on
was dual-licensed. This allowed usage of the code under any of the
licenses (which including LGPL):

"""
* Alternatively, the contents of this file may be used under the terms of
* the GNU Lesser General Public License Version 2 or later (the "LGPL"),
* in which case the provisions of the LGPL are applicable instead
* of those above. If you wish to allow use of your version of this file only
* under the terms of the LGPL, and not to allow others to
* use your version of this file under the terms of the MPL, indicate your
* decision by deleting the provisions above and replace them with the notice
* and other provisions required by the LGPL. If you do not delete
* the provisions above, a recipient may use your version of this file under
* the terms of the MPL or the LGPL.
"""

When refactored (leading to the creation of this new plugin), I chose all
new code to be LGPL-only (which was allowed for pre-existing code) by removing
the MPL sections.

The headers were all updated, but not the plugin license field. This commit
fixes this.

tests: insertbin: update tests to 1.x scenario

The thread that calls the success/failure callback can be the
same that is adding/removing the element as the IDLE probe can
happen instantly if the pad is not 'busy'.

This required moving some checks for the callback counter around
as well as removing some pad pushes from the main test thread as
they were made useless after the IDLE pad probe was fixed in core
by commit 0324358ebc8122a41ab499a3ef7973b61c3e621b

tests: camerabin: avoid warning if test is failing

In case the test fails the message won't be returned and it
will lead to an assertion that might be misleading to those
debugging the issue.

wrappercamerabinsrc: stop source to do internal reconfiguration safely

In order to be able to change the caps on multiple capsfilters the
source element needs to be stopped, otherwise it will get a few
reconfigure events and might try to renegotiate while the bin
is still transitioning its caps, leading to a not-negotiated failure
and the image capture won't happen because the source will be
unusable.

The solution is to keep the source in paused while the caps are being
changed in the bin, and then bring the element back to playing once
it is done. Unfortunately this increases the image capture latency,
but it should always work.

A possible improvement to reduce the latency is to add another signal
to be called before 'start-capture': 'prepare-capture'. At this step
the camera source should set all caps it needs and get the source
ready for doing the capture as soon as 'start-capture' is called.
This can be done on a future commit

camerabin: remove unused variable

Probably a left over from 0.10 -> 1.x porting

mpg123: improved error report and checks

Signed-off-by: Carlos Rafael Giani <dv@pseudoterminal.org>

uridownloader: Fix race condition between EOS handling and downloading a range

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

aiffparse: adaptive buffer size

Copied from wavparse, helps with CPU usage on high bitrate
files.

Since b1a756fd, applemedia fails to build on osx 10.7, which lacks VideoToolbox.h... so disable it on OSX if that header is missing.

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

aiffparse: fix data start offset calculation not counting COMM size

In particular, this fixes seeking back to the start reading data
out of sync with sample start granularity, yielding swapped channels.

tests: gdpdepay: update to 1.x events

Tests needed stream start at beginning and segment after the
caps event

gdp: add DTS to headers

gdppay: update to 1.x reality

* stream-start-id is mandatory at the beginning, so add that to the
gdp headers
* caps must be sent before new segment, invert the order from legacy
0.10 code

And fix the tests as a ref is now kept for those buffers that compose
the header

fisheye: Fix documentation

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

h264parse: Default framerate to 0/1 when no informations found

It is not perfect but it allows us to be sure that the mandatory 'framerate'
field is present in the caps.

As soon as some information is found in the stream, that will be
updated.

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

tests: gdppay: refactor and add missing bits

Most of the tests weren't updated after the sticky events order
and stream start. Fix that and refactor those tests check that
are the same to some common functions.

Those functions still don't actually test the content but at
least now they are in a single place and can be improved
without replication

tests: gdppay: Do not forget to send stream-start event on test

Even if we don't want to push the caps event, a stream-start must
be present before any data in 1.x

gdppay: push the stream-start event

instead of just swallowing upstream's stream-start into its
payload, push it as elements expect a stream-start before data

tsdemux: We do not support the SKIP seek flag but that is not a problem

Automatic update of common submodule

From d48bed3 to 1a07da9

tests: camerabin: improve preview handling in tests

Always erase the preview image data to allow the next check for
it to operate properly

uridownloader: Always release the object lock in gst_uri_downloader_fetch_uri_with_range()

even if downloader->priv->urisrc is not set. This avoids deadlock under some
some conditions on cancel.

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

opencv: The plugin works with 2.4.8 too

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

codecparsers: Don't use gst_ as prefix for private symbols

Otherwise they get exposed/accessible in the resulting .so

And make check-exports will complain

tsdemux: Fix flags comparison operator precedence

Fix operator precedence, so the whole operation is
negated instead of only flags variable.

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

mpegts: install but nodist for generated header

androidmedia: Don't handle FLUSHING or NOT_LINKED as error

And also just stop the task for FLUSHING.

mxfdemux: Don't go into pull mode when the sequential flag is set

When the scheduling query results has GST_SCHEDULING_FLAG_SEQUENTIAL set
in its flags don't go into pull mode to prevent over-eager seeking.

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

tests: h264parse: fix tests after leading 0x00 removal

Commit 6af387cd5ab2c946025e5499903e75ee87b063a9 made h264parse
strip a leading 0x00 byte from some output scenarios. This broke
tests as bs_to_nal test expects one more byte on the output.

Fix this by comparing the output with the expected stripped version,
too.

codecparsers: refactor common nal parsing to nalutils

Moves common code from h264 and h265 to a separate file

h264parse: Fix multiple SEI messages in one SEI RBSP parsing.

An SEI RBSP could contains more than one SEI message as specified in
7.4.2.3.1.

This commit change the parser API: the gst_h264_parser_parse_sei()
function now create and fill a GArray containing GstH264SEIMessage.

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

h265parser: remove trailling 0x00 bytes as the spec doesn't allow them

Just like h264, the spec doesn't allow the last bytes of a NAL to be 0x00.
So remove it as it is probably just padding.

This is related to https://bugzilla.gnome.org/show_bug.cgi?id=721384

h264parser: remove trailling 0x00 bytes as the spec doesn't allow them

The spec states that the last byte of a NAL 'shall not' be 0x00
and it is allowed for byte-stream format to add padding 0x00 for
alignment.

So our parser should strip any trailling 0x00.

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

mpegtsmux: don't leak pad name when treating events

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

mpegpsmux: update last_ts to mux correctly

We use last_ts to decide what buffer to mux next so make sure that it
always contains the last known valid timestamp.

h265parser: Initialize pointer correctly that is never assigned but freed in error cases

Fixes crash on broken streams.

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

d3dvideosink: First destroy the window, then unregister the class

It's impossible to create another pipeline with d3dvideosink after disposing
the previous one due to some problem in d3dvideosink. The message is: "Unable
to register Direct3D hidden window class".

I've evaluated the problem and it's that UnregisterClass() in working thread is
called before DestroyWindow() and UnregisterClass() does nothing.

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

mpegpsmux: fix timestamp handling

If the first buffer that we handle for a stream has no timestamp, we
would never consider this pad again for muxing which causes queues to
fill up and pipelines to stall. Instead, try to mux pads with -1
timestamps as soon as possible.

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

mpegpsmux: return GST_FLOW_EOS when we pushed EOS

Return GST_FLOW_EOS from the collected function when we pushed EOS or
else it simply keeps on calling our collected function.

applemedia: Actually the enc/decbins are not used anywhere currently

Code is commented out.

applemedia: Fix build on iOS and probably also on OSX

The video toolbox is not available on iOS, and also on OSX
we should probably build all source files that are used for
it.

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

h265parse: Update caps when receive VPS/SPS/PPS

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

h265parser: Fix segfault when parsing HRD parameter

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

openexrdec: Don't start scanning for the header from the start again each time

This will be incredible slow if the upstream block size is very small. Instead
continue scanning for the header where we previously stopped.

For the standard filesrc block-size this made decoding a file about
3 times faster.

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

sfdec: This is a decoder, not a demuxer

Don't confuse autopluggers!

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

tsdemux: Fix leak of PCROffsetGroup

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

audiomixer: Remove some useless assignments and skip the current part of the buffer if the pad is muted

resindvd: Merge changes across from master mpegdemux

Merge various changes and fixes from the master mpegdemux
Performance improvement from the way streams are organised,
return flow combining, language tag event generation,
adjustments and fixes in debug output, and things like that.

mpegdemux: Merge some changes from the resindvd copy

One bug fix, some debug changes and other inconsequential changes
just to reduce the diff between the 2 copies a bit.

audiomixer: Fix and simplify overlap calculation

resindvdsrc: fix minor memory leak when creating title info message

amcvideodec: Add the divx variant for mpeg4 video

Looks like all the mpeg4 video decoders also accept the divx
variants. So we better add those caps too

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

configure: fix (again) check for DB API: 5.4 is Not Recent Enough

Maybe testing the version is clearer, but testing for < 5 is not
enough, my version is 5.4 and does not yet have those new enums.

If you git blame to this and have a version > 5.4 that does not
either, please feel free to join along and bump the version.

h265parser: Fix segfault when parsing VPS

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

faceblur: set maximum feature size to 0x0

Previously faces would only be detected if they were at least 30x30 pixels
large and at most 32x32 pixels. We keep the minimum setting (maybe needs
a property as in facedetect) but disable the maximum feature size.

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

facedetect: set maximum feature size to 0x0

This disables the "max feature size" feature. The current configuration
is totally busted: The max feature size is hard-coded to 2 pixels more
than the user-supplied min feature size which pretty much means you need
to guess the size of the person's face to within a few pixels to get the
code to find it.

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

tsdemux: fix memleak of PCROffsetCurrent

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

h265parser: prevent to overrun chroma_weight_l0_flag

The index of elements cannot exceed or equal to the size of elements.

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

schroenc: set buffer offset to buffer timestamp

oggmux expects this to order incoming buffers

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

dvb: Use DVB_API_VERSION to know if we have recent enough version

We end up with the same end-result but it's more comprehensible

configure: check for good enough dvb kernel headers

Recent changes are using new dvb api. Don't built the plugin, if we can't.
Fixes #721869

aiffparse: misc code cleanups

Fix the launch examples for 1.0. Turn more logging into _OBJECT variants.

dashdemux: remove unnecessary variable

Instead of using 2 indexes that were actually the same, use only
one. Saves a variable.

dashdemux: fix seeking when the mpd uses a segment template

Remove the dashdemux seeking function to use the one implemented
in mpdparser as it is more complete. This also makes dashdemux not
crash when seeking on streams that use segment templates.

mpeg4videoparse: do not lose DISCONT flag if buffer isn't pushed

mpeg4videoparse might not push buffers while parsing. If those buffers
contain the DISCONT flag, it gets lost and downstream won't get any
buffer with the flag.

Fix it by adding the DISCONT to the next pushed buffer.
This makes backwards playback work.

sfdec: skip '\0' strings for metadata

libsndfile does not filter empty strings. We get a warning from gstreamer when
setting this as a tag.

sfdec: break long method

Extract taglist creation into separate funtion.