h264parse: Don't duplicate SPS/PPS per config-interval if exists

Don't need to manually insert SPS/PPS since inband data could be useable.

Fixes #824

meson: build opencv and ipcpipeline examples

https://gitlab.freedesktop.org/gstreamer/gst-plugins-bad/issues/576

cc708overlay: fix deadlock

We would forget to unlock when a caption data buffer is deemed
out of segment, which makes everything lock up next time buffers
are received.

Run gst-indent through the files

This is required before we enabled an indent test in the CI.

https://gitlab.freedesktop.org/gstreamer/gstreamer-project/issues/33

webrtc: Remove duplicate declarations

This causes 'redefinition of typedef ...' errors on GCC 4.5.3

hlssink2: Fix string leak

Need to free allocated string memory on _finalize()

msdk: remove unnecessary assignment

CodecProfile will be set in MFXVideoDECODE_DecodeHeader() to match
the input stream. Setting the hard-coded profile here will mislead
user that msdkh265dec supports a special profile only.

msdk: update the sink and src caps of msdkh265dec

Now hevc 10bit video can be decoded correctly, so update the sink
and src caps accordingly.

This fixes https://gitlab.freedesktop.org/gstreamer/gst-plugins-bad/issues/720

msdk: set right BitDepth and Shift for P010 mfx frame

BitDepth is 10 and Shitf must be set to 1 when creating P010 mfx
frame in MSDK

msdk: don't use hard-coded video format

Some codecs may support varied formats, e.g. HEVC may support NV12
and P010_10LE etc

msdk: adjust the RT format for P010 surface

According to VA API, VA_RT_FORMAT_YUV420_10 is expected for P010
surface

msdk: create VA_FOURCC_P010 surface from dmabuf

msdk: VA_FOURCC_P010 frame lock

P010 and NV12 have the same layout, so we may reuse the code in
gst_msdk_frame_lock()

msdk: map MFX_FOURCC_P010 to VA_FOURCC_P010

msdk: make sure the surface data is set for GST_VIDEO_FORMAT_P010_10LE

P010_10LE and NV12 have the same layout, so we may reuse the code.

msdk: map GST_VIDEO_FORMAT_P010_10LE to MFX_FOURCC_P010

msdk: add an assert in gst_msdk_frame_lock() for unhandled formats

We will add support for more formats, e.g. P010, Adding an assert
here may catch the error early

msdk: use separate src caps for msdkh265dec

We will add more formats in the src caps of msdkh265dec, so let
msdkh265dec uses separate src caps. Note it doesn't change any
capability

msdk: Make sure a variable is initialized when it is used

Previously alloc_info is initialized when both thiz->initialized
and thiz->allocation_caps are true, but only thiz->initialized is
checked when alloc_info is used.

msdk: Close fd handle

Otherwise it will result in resource leak if mem == NULL

opencv: define CASCADE_DO_CANNY_PRUNING for opencv < 4

opencv: make compatible with opencv 4

Closed #826

opencv: fix indentation

configure.ac: add gst-allocator check in msdk.

With out check gst-allocator, there will be a gst-allocator api
symbol link error when call for libgstmsdk.so.

decklink: fixup internal time tracking over buffering pauses

Instead of relying on buffers after a state change to PLAYING to always start
from 0, track the amount of time we have spent outside playing but not changed
state to PAUSED.

tests/webrtc: fix tests for no libnice

webrtcbin will fail the state change to READY when libnice elements are
not available.

tests/webrtc: use the existing functions in the plugin

Instead of redefining our own, use the function implementations in
webrtcsdp.c and utils.c

webrtc: fix typo in RTCRemoteOutboundRTPStreamStats

webrtc: add a few comments on bundle and src pad exposure

webrtcbin: factor out dtls fingerprint setting

webrtc: remove extra 'pad' from log line

webrtc: move some functions to the appropriate files

meson: Fix Windows CUDA dependency check

Python returns 'None' string for unknown environment

nvh265enc: Fix email address typo

nvenc: Fix undefined reference build error on MSVC and CUDA 9.1

nvenc: Fix MSVC build error C2121

nvenc: Add meson build with Windows support

Note that, since Nvidia does not provide nvEncodeAPI.lib file,
find_library() couldn't be used for build on Windows.
This patch changes to load nvEncodeAPI(64).dll or libnvidia-encode.so
in runtime

nvdec: Add meson build with Windows support

nvdec: Drop dynlink interface and use NVIDIA CODEC SDK instead

dynlink_* was introduced since CUDA Toolkit 9.x but it's deprecated from 10.0.
Instead of using #ifdef hack, shipping nvidia headers of NVIDA CODEC SDK
can make build/code simple

nvdec/nvenc: Shipping NVIDA Codec SDK headers

Add cuvidec.h, nvcuvid.h and nvEncodeAPI.h of NVIDIA Codec SDK 8.2.16

nvenc/nvdec: Add support CUDA Toolkit 10.0

examples: codecparser: fix compiler warnings for unstable API use

Breaks build bots and build with --werror

examples: force gtk to x11 backend

Until someone makes it work with the wayland backend.
The code currenty assumes and hard-codes X11.

https://gitlab.freedesktop.org/gstreamer/gst-plugins-bad/merge_requests/49

meson: build more examples

See https://gitlab.freedesktop.org/gstreamer/gst-plugins-bad/issues/576

meson: Specify encoding to UTF-8 when building with MSVC

Use build arguments consistent with core and -base. This can also
remove noisy "C4819" warning of non-us locale MSVC.

msdk: don't declare headers in meson

This partially reverts commit 0bae64835302574953c3e5e89fa0822eb4f6ca4b.

The compiler tells ninja the header dependency, then there is
no need to add them explicity.

msdk: declare headers in meson

msdkdec: move output-order out of decode bass class.

Since output-order is a deprecated attribute, move it out of decode
bass class and configure it in each sub decoder class who need it.

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

msdk: add support for open sourced MediaSDK

MediaSDK has been released as open source [1], but the directories
where it installs its files, are different from the binary only
distribution.

This patch adds to the libraries path the directory /lib. Also it
is defined in meson if the include directory has the mfx/ prefix,
something that is already handled in autotools.

1. https://github.com/Intel-Media-SDK/MediaSDK

Webrtcbin : Need to use 'host' from gst_uri_get_host s libnice agent expects it

Currently master code of gst1-plugins-bad use plain-string host name while passing it to
libnice agent: nice_agent_set_relay_info() in gstwebrtcice.c while adding turn_server(_add_turn_server).

It is observered that if we don't convert the host parameter by using gst_uri_get_host, it fails in libnice agent(0.1.14-1).

Code does, actually, set the host correctly but while passing params to nice_agent_set_relay_info, it uses incorrect one.

https://gitlab.freedesktop.org/gstreamer/gst-plugins-bad/issues/823

siren: Fix floating point invalid operation

Mix of single and double precision leads to zero-by-zero divide
for upper 64-bit of the xmm register, even though they are not
used.

dtls: Fix compilation without deprecated APIs on OpenSSL 1.1.x

cameracalibrate: Fix build error on Mac OSX

It fails to build only on Mac OSX with the following error.

In file included from ../subprojects/gst-plugins-bad/ext/opencv/gstopencv.cpp:45:
../subprojects/gst-plugins-bad/ext/opencv/gstcameracalibrate.h:96:38: error: a space is required between consecutive right angle brackets (use '> >')
  std::vector<std::vector<cv::Point2f>> imagePoints;
                                     ^~
                                     > >
1 error generated.

Fix: #817

meson: Fix invalid keyword warning

"required" keyword is not a valid argument for has_header()

WARNING: Passed invalid keyword argument "required".
WARNING: This will become a hard error in the future.

decklinkvideosink: Cast result of bitwise-or of multiple enum values to the enum again

In C++ the bitwise-or results in an int, and ints are not implicitely
cast to enums.

See https://gitlab.freedesktop.org/gstreamer/gst-plugins-bad/merge_requests/25#note_78122

d3dvideosink: Remove unused variable

Fixes a compiler warning.

decklinkvideosink: Add support for outputting closed captions

Add Gitlab CI configuration

This commit adds a .gitlab-ci.yml file, which uses a feature
to fetch the config from a centralized repository. The intent is
to have all the gstreamer modules use the same configuration.

The configuration is currently hosted at the gst-ci repository
under the gitlab/ci_template.yml path.

Part of https://gitlab.freedesktop.org/gstreamer/gstreamer-project/issues/29

build: link against the gnustl dependency on android for c++ plugins

avdtputil: Remove dead code

These function are not used at all, using them together with the
transport-volume property from avdtpsrc may end up in a binding loop so
we better remove the functions.

avdtpsrc: Fix transport-volume write

If properties are proxied through GBinding this can work only if the
proxied property keeps it's own value. The previous implementation will
read the original value if the proxied property signals a change and
thus nothing will happen.

srtp: Add unit tests for MKI support

Both as a srtpenc ! srtpdec pipeline and as a test that uses
fake packets for multiple mixed MKIs.

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

srtpenc: Add support for adding a MKI

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

srtpdec: Add support for MKI

Add support for MKIs which allow multiple keys to be used with a single SRTP stream.

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

dshowsrcwrapper: add implementation of GstDeviceMonitor

Right now this only works for video. An attempt was made at adding
monitoring following the example winks, but it seems the only devices that
can be easily detected are KS sources, which winks already handles.

dshowsrcwrapper: refactor device selection, filter creation, and caps retrieval

This allows a future GstDeviceProvider to more easily query devices and caps.

dshowsrcwrapper: add get_property implementation to sources

dshowvideosrc: handle empty strings for device and device-name

The device and device-name properties should treat NULL and empty strings the same

dshowvideosrc: delay selecting device until source is started

The previous behaviour had issues when setting one of the device properties
after _get_caps had been called. The device shouldn't be locked in until after
_start has been called.

dshowvideosrc: fix template caps to reflect actual supported caps

dshowsrcwrapper: add some helpful debug statements

ksvideosrc: initialize pointer to satisfy MSVC

ksvideosrc: support 'Y16 ' FOURCC as GRAY16_LE

The FLIR Boson USB provides 16-bit grayscale video using this FOURCC.

ksvideosrc: gracefully handle device prop when empty string

mpegts: Register a number of new boxed types to support auto generation of bindings.

decklinkvideosrc: Search from line 1 again if we didn't find closed captions at the same line as before

tsdemux: implement preliminary support for the bitrate query

Return the size / total duration as a ballpark estimate.

https://gitlab.freedesktop.org/gstreamer/gst-plugins-base/issues/60

h265parser: Various update of vps parsing

* Add FIXME for future correction of HRDParames parsing.
Spec. defines that the number of HRDParames could be up to
"vps_num_layer_sets_minus1 + 1" (i.e., 1024).

* Add parsing vps_base_layer_{internal,available}_flag.

* Fix possible invalid vps_extension parsing.

Fixes #798

tests: h265parser: Add test parsing nonzero vps_max_layer_id in VPS

h265parser: Fix wrong maximum range check in gst_h265_parse_vps()

Fix validation of some syntax.

Fixes #798

meson: avoid unkown variables in the msdk tests

dtls: Link against winsock2 on Windows

Should fix the Windows build after
b1509b1047bb76c9b2d8b14e9cecd0da72fd8e65.

decklinkvideosink: fix segfault when audiosink is closed before videosink

dtlsconnection: Print out errno info for syscall errors

As suggested in [the SSL_get_error manpage][1]. Upgrade the message to a
warning if the errno isn't 0 (success). The latter apparently means the
transport encountered an EOF (shutdown) without the shut down handshake
on the (D)TLS level. This happens quite often for otherwise normal DTLS
connections.

[1]: https://www.openssl.org/docs/man1.1.1/man3/SSL_get_error.html

dtls: Properly display all errors/warnings from ERR queue

Print out all errors from the OpenSSL error queue instead of just
looking at the topmost error. Using the callback interface also removes
the need for formatting using a buffer on the stack.

dtlsconnection: Replace if-else chain with switch

A more idiomatic construct for testing a bunch of integers.

dtlsconnection: Clear error queue before SSL_do_handshake

As documented on [the SSL_get_error manpage][1] we need to empty the
error queue before making any call that we check with SSL_get_error.

[1]: https://www.openssl.org/docs/man1.1.1/man3/SSL_get_error.html

dtlsconnection: Lower BIO error log level to DEBUG

Periodic SSL_ERROR_SYSCALL errors with errno == 0 seem to be normal
behavior for DTLS connections.

Fixes https://gitlab.freedesktop.org/gstreamer/gst-plugins-bad/issues/677

dtlsconnection: Run gst-indent

cccombiner: Add unit test

cccombiner: Add new closed caption combiner element

This is the inverse of ccextractor: it takes a video and closed caption
stream and places the closed caption metas on the video output buffers.

ccextractor: Handle multiple caption metas per buffer

msdk: fix intel sdk libdir path

the 2018.3.1 intel sdk release places libraries into /lib64 instead of
/lib/lin_x64 or /lib/x64, this commit adds /lib64 to the libdir
locations list

Fixes #815

d3dvideosink: Fix calculating buffer size of packed format

The Pitch (i.e., stride) of D3DLOCKED_RECT structure is already
respecting required stride value.

d3dvideosink: Set actual buffer size to buffer pool

If configured size of buffer pool and actual buffer size are mismatched,
pool will do unwanted buffer resize.

Fixes #801

d3dvideosink: Use lock/cond to wait internal window construction

We can save one (or possibly more) sleep time by using lock/cond

d3dvideosink: Don't leak GThread object

Need to call g_thread_join() to avoid leak

d3dvideosink: Print lock/unlock log with trace debug level

d3dvideosink: Remove white space

h265parser: Stop considering 6 bytes NAL complete

This is only TRUE for EOS / EOB NALs. Before this patch, passing
any valid 6 bytes of a NAL would make the parser pretend this NAL is
complete.