msdk: release resources if failing to create the buffer pool

Needn't check the pointers against NULL because the pointers are
non-NULL

msdk: meson: compile vp9dec if available

based on the patches provided by Haihao Xiang <haihao.xiang@intel.com>

msdk: add mfx in include path

Thus removing the preprocessor's directives to included if found.

msdk: meson: use libmfx pkg-config if available

Refactoring to bail out early if MediaSDK is not found.

based on the patches provided by Haihao Xiang <haihao.xiang@intel.com>

msdk: add in configure more directories to search for libraries

msdk: don't reset the external frame allocator

In gst-msdk, a mfx session may be shared between different gst
elements, each element tries to set the frame allocator. However, per
the MSDK documation[1], the behavior is undefined if reset the frame
allocator while the previous allocator is in use. Fortunately all
elements use the same frame allocator, so we can avoid to call
MFXVideoCORE_SetFrameAllocator again.

[1]: https://software.intel.com/en-us/node/628430#MFXVideoCORE3

msdkvpp: don't use NV12 as default output in normal mode

If so, BGRA is the preferred output format hence BGRA will be selected
as input format by default, e.g. in the pipleline below, BGRA instead of
NV12 is selected without renegotiation, so we can avoid the NV12 issue
(see commit 3f2314a) by default.

gst-launch-1.0 videotestsrc ! msdkvpp ! glimagesink

msdkvpp: close the current VPP session if this session has been initialized

Otherwise MFXVideoVPP_Init will fail because it is called twice without
a close.

Example pipeline:

gst-launch-1.0 videotestsrc ! msdkvpp ! glimagesink

Sometimes glimagesink emits GST_EVENT_RECONFIGURE event which results
in that MFXVideoVPP_Init is called twice, then get the negotiation
failure below:

0:00:00.093715518 21218 0x558ef56231e0 ERROR                msdkvpp
gstmsdkvpp.c:995:gst_msdkvpp_initialize:<msdkvpp0> Init failed
(undefined behavior)
WARNING: from element /GstPipeline:pipeline0/GstMsdkVPP:msdkvpp0: not
negotiated

After applying this commit, the pipeline above may run without
negotiation failure, however NV12 layout in dmabuf mode is selected in
renegotiation, the display image is corrupted due to the NV12 issue which
was mentioned in commit 3f2314a. Some other fixes are needed to avoid
renegotiation by default

msdkvpp: don't update pads' info if nothing have changed

msdkenc: break out of flush frames loop on error

In general, we should assume any unhandled error is
non-recoverable.

In the flush frames loop, some error states can cause us
to never increment the task and therefore we get stuck
in an infinite loop and generate GST_ELEMENT_ERROR
over and over again.  This eventually consumes all
system memory and triggers OOM.  Thus, assume the worst
and break out of the loop upon the first "unhandled" error.

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

wasapi: Fix double call to Start when resetting the element

When either the source or sink goes from PLAYING -> NULL -> PLAYING,
we call _reset() which sets client_needs_restart, and then we call
prepare() which calls IAudioClient_Start(), so we don't need to call
it again in src_read() or sink_write(). Unlike when we're just going
PLAYING -> PAUSED -> PLAYING.

wasapisink: Don't call CoUninitialize() twice in unprepare()

It has to be symmetric with CoInitialize(), otherwise everything else
will fail.

decklinkvideosink: Don't forget to unref clock after usage

And don't unref a clock that is potentially NULL.

webrtc: include stdlib.h for atoi()

Fixes #857

tests: Enable more tests on Windows

tests: Remove compositor test

It's moved to -base and no more part of -bad

tests: Remove pointless unistd include

Remove openglmixers plugin, moved to -base

Merged into the existing opengl plugin in -base.

Remove compositor plugin which was moved to -base

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

Remove GstVideoAggregator, moved into libgstvideo in -base

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

opencv: fix disting of new header files

Fixes distcheck.

stereo: remove plugin which has been merged into audiofx in -good

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

msdk: Fix invalid return type build warning

void function cannot have return value

gst-plugins-bad/sys/msdk/gstmsdkbufferpool.c(332): warning C4098:
'gst_msdk_buffer_pool_release_buffer': ...

openal: Fix AL header path for Mac OS X

OpenAL.framework doesn't provide 'alext.h'.
Thus, openal is enabled only if the dependency
is set by pkg-config.

fixes: #852

player: Logging fix for updated audio info

Bitrate was logged a second time instead of the max_bitrate.

timecodestamper: Don't use deprecated API

webrtc/receive-bin: block pads before dtlssrtpdec:

Fixes SSL errors in fast-start scenarios and whenever media stream may
be received before an answer is set.

webrtcbin: only change the receive state after setting the dtls-client

Doing so before will cause SSL errors with fast-start implementations
like Chrome or if media data arrives before an answer.

webrtc: A couple of documentation fixes

set-*-description only takes the combined GstWebRTCSessionDescription
object

decklinkvideosink: Fix support for raw CEA608 input and add support for raw CEA708 input

The former was only considering the first byte pair, for the latter we
have to convert raw CEA708 cc_data into CDP.

ccconverter: Add unit test

closedcaption: Add ccconverter element that can convert between different types of Closed Caption formats

Fix build when we have tinyalsa but not its header

tests: Re-enable isoff test on Windows

... and drop libxml2 dependency since it never be used by
both test and gstisoff.

tests: Fix build with MSVC

meson: Prefer to use join_paths() over '/'

... to avoid mixing '/' and '\' in a path string on Windows.

tests: Enable unit test on Windows

Allow run some unit tests on Windows.
* Add dependency explicitly for some test cases, otherwise plugins couldn't be
  loaded on uninstalled environment of Windows.
* Add missing GST_PLUGIN_LOADING_WHITELIST on meson build.

kmssink: Accept underscore in property

Otherwise sdi_mode and similar properties would be canonicalised to
sdi-mode.

kmssink: fix tmp_kmsmem leaks

configure_mode_setting() keeps a ref on tmp_kmsmem which is released in
gst_kms_sink_show_frame().
But if for some reason configure_mode_setting() is re-called before
showing a frame or if none is showed this memory was leaked.

Remove acmmp3dec and acmenc plugins

ACM is an ancient legacy API, and there's no point in
keeping it around for a licensed mp3 decoder now that
mp3 patents have expired and we have a decoder in -good.

We didn't ship this in cerbero anyway. If there's a good
case for the AAC encoder (which is LC only anyway) someone
should write a new plugin based on current APIs, that can
actually be built out of the box.

Fixes #850

meson: fix opencv=disabled case if opencv is available on the system

ext/opencv/meson.build:103:2: ERROR:  Unknown variable "gstopencv_dep".

meson: wildmidi detection fixes

Need at least 0.4.2 apparently.

ext/wildmidi/meson.build:11:2: ERROR:  String '' cannot be converted to int

curlhttpsink: free content type before assigning it

Not doing this would result in a memory leak.

curlbasesink: cleanup transfer immediately when done

fdkaac: Syntax tweaks to increase readability

Mostly move some assignments out of conditionals that gst-indent
completely destroys.

fdkaacdec: Enable 8-channel playback

The decoder seems to default to 6 channels max, downmixing 7.1 to 5.1.
Disable the channel limit to expose all channels to GStreamer.

In addition, none of the standard configurations use ACT_SIDE channels.
The rear channels of the 7.1 configuration have to be taken from
ACT_BACK. See the table in aacenc_lib.h, reproduced here:

----------------------------------------------------------------------------------------
 ChannelMode           | ChCfg | Height | front_El      | side_El  | back_El  | lfe_El
-----------------------+-------+--------+---------------+----------+----------+---------
MODE_1                 |     1 | NORM   | SCE           |          |          |
MODE_2                 |     2 | NORM   | CPE           |          |          |
MODE_1_2               |     3 | NORM   | SCE, CPE      |          |          |
MODE_1_2_1             |     4 | NORM   | SCE, CPE      |          | SCE      |
MODE_1_2_2             |     5 | NORM   | SCE, CPE      |          | CPE      |
MODE_1_2_2_1           |     6 | NORM   | SCE, CPE      |          | CPE      | LFE
MODE_1_2_2_2_1         |     7 | NORM   | SCE, CPE, CPE |          | CPE      | LFE
MODE_6_1               |    11 | NORM   | SCE, CPE      |          | CPE, SCE | LFE
MODE_7_1_BACK          |    12 | NORM   | SCE, CPE      |          | CPE, CPE | LFE
-----------------------+-------+--------+---------------+----------+----------+---------
MODE_7_1_TOP_FRONT     |    14 | NORM   | SCE, CPE      |          | CPE      | LFE
                       |       | TOP    | CPE           |          |          |
-----------------------+-------+--------+---------------+----------+----------+---------
MODE_7_1_REAR_SURROUND |     0 | NORM   | SCE, CPE      |          | CPE, CPE | LFE
MODE_7_1_FRONT_CENTER  |     0 | NORM   | SCE, CPE, CPE |          | CPE      | LFE
----------------------------------------------------------------------------------------
- NORM: Normal Height Layer.     - TOP: Top Height Layer.  - BTM: Bottom Height Layer.
- SCE: Single Channel Element.   - CPE: Channel Pair.      - LFE: Low Frequency Element.

Restores 8 channels to
https://www2.iis.fraunhofer.de/AAC/7.1auditionOutLeader_v2_rtb.mp4

fdkaacenc: Add support for v2.0.0 and its configurations

These are MPEG-standard configurations so should be checked first,
before the PCE clones added by v0.1.4.

fdkaacenc: Add support for MPEG configuration 7

fdkaacenc: Add a description of each configuration

fdkaacenc: Prefer REAR for surround channels

We (currently?) prefer to use the REAR positions for what MPEG calls
"surround" channels.

Remove daala plugin

Not so useful now that all effort is focused on AV1 instead.
Was only ever enabled with --enable-experimental anyway.

Fixes #848

meson: Add libsinc to the CFLAGS

Otherwise gst-i18n-plugin.h can't be found.

meson: build openni2 plugin

meson: build openmpt plugin

meson: build openexr plugin

meson: build openal plugin

meson: build sndfile plugin

meson: install freeverb preset file

meson: build voamrwbenc plugin

meson: build teletext plugin

meson: build modplug plugin

meson: build wildmidi plugin

meson: build dc1394 plugin

meson: build ofa plugin

meson: build gme plugin

meson: build vdpau plugin

meson: build tinyalsa plugin

meson: build vcdsrc plugin

Remove spc plugin

Replaced by gme plugin. Fixes #467

closedcaption: Use new API for converting GstVideoCaptionType from/to GstCaps

closedcaption: Replace GST_VIDEO_CAPTION_TYPE_CEA608_IN_CEA708_RAW with CEA608_S334_1A

As a side-effect we can now actually store the line offset in the
line21dec element, and have to perform fewer transformations in the
decklink elements (which were also buggy as they assumed a single byte
triplet per meta).

line21dec: If CCs are not found at the previous line, reset and search from the beginning again

line21dec: Don't read more lines than the height of the frame

removesilence: add libm to meson.build

removesilence: reset filter on start and on segment events

removesilence: improve documentation

removesilence: add threshold property

silence thresold was hardcoded to -60 dB, now it is configurable
using this new property

Closes #63

removesilence: add properties to detect silence only after minimum silence time/buffers

Closes #63

removesilence: add silent property to control bus message notifications

Closes #63

removesilence: post bus messages when silence is detected/finished

Closes #63

removesilence: add squash property

allows to output buffers without timestamp gap when silence is removed

Closes #63

sys: applemedia: meson: Add dependencies by using appleframeworks

gst-build#13

waylandsink: Avoid race condition on multi-threaded client

When waylandsink is used on some other thread than the main wayland
client thread, the waylandsink implementation is vulnerable to a

condition related to registry and surface events which handled in
seperated event queue.

The race that may happen is that after a proxy is created, but
before the queue is set, events meant to be emitted via the yet to

set queue may already have been queued on the wrong queue.

Wayland 1.11 introduced new API that allows creating a proxy
wrappper which can help to avoid this race condition.

examples/wayland: Add wayland multi-thread client example

This is for testing race condition with multi-thread wayland client
environment. The race condition will be resolved with wayland proxy
wrapper API when handling event queue.

h26{4,5}parse: Don't confuse nal of codec_data with frame

vps/sps/pps in codec_data shouldn't be considered as inband data.
Otherwise, h26{4,5}parse never insert them to nal when transform
(packetized to byte-stream) use case

nvdec: Rely on upstream's value for interlace-mode with hevc

The nvdec API doesn't seem to produce interlacing information with hevc
streams so rely on upstreams value for interlace-mode

msdk: change the wait time for encoder and vpp

In MSDK samples, the wait time for encoder, decoder and vpp is
300000. Let's set the wait time to the same value in msdk plugin

msdk: correct the error message

It is VPP session instead of Encoder session

msdk: check the created context against NULL pointer

gst_msdk_context_new_with_parent() may return NULL

msdk: decrease the reference count of object

Otherwise there are reference leaks if failed to clone or join a
mfx session

msdk: assign the returned value to status variable

Othervise the subsequent check will use the stale value of status
variable

msdkh265enc: output main-10 bitstream if the input is P010_10LE

Tested on KBL using the following command:
gst-launch-1.0 videotestsrc num_buffers=100 ! video/x-raw,format=P010_10LE ! \
msdkh265enc ! filesink location=a.hevc

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

msdkh265enc: add P010_10LE to the sink pad template

msdkh265enc: re-add the sink pad template

We will add more profiles in the sink caps of msdkh265enc, so let
msdkh265enc re-add the sink pad template. Note this change doesn't
impact any capability

msdkenc: handle P010_10LE input format

Note it is up to each codec to support P010_10LE format

msdkenc: use macro GST_VIDEO_INFO_FORMAT if possible

decklink: calculate the decklink output time from the internal clock

Fixes the time calculations when dealing with a slaved clock (as
will occur with more than one decklink video sink), when performing
flushing seeks causing stalls in the output timeline, pausing.

Tighten up the calculations by relying solely on the internal time
(from the internal clock) for determining when to schedule display
frames instead attempting to track pause lengths from the external
clock and converting to internal time.  This results in a much easier
offset calculation for choosing the output time and ensures that the
clock is always advancing when we need it to.

This is fixup to the 'monotonically increasing output timestamps' goal
in: bf849e9a69442f7a6f9d4f0a1ef30d5a8009f689

tests: mpegvideoparse: add unit test for CEA-708 closed captions extraction

mpegvideoparse: extract CEA-708 closed captions

mpegtsmux: Change unit test to not generate zero-sized buffers ever