docs: fix typo s/abonormally/abnormally/

docs: fix typo s/incomming/incoming/

rtp: fix indentation after G_DEFINE_TYPE

A missing colon after G_DEFINE_TYPE declaration was confusing gst-indent
and causing problem in the pre-commit hook.

Add the missing colon and fix the following function declaration to
follow the normal GStreamer style.

rtpsession: fix comment to refer to buffers instead of groups

One comments in gst_rtp_session_chain_send_rtp_common() is referring to
groups in a buffer list, however this concept of "group" comes from
GStreamer 0.10 and does not exist anymore in GStreamer 1.0, so update the
comment to refer to buffers instead.

rtpsource: add comment to explain why probation queue is not always cleared

test: rtpbin_buffer_list: add test to verify that stats are correct

Add a test to verify that stats about sent and received packets are
correct even when using buffer lists.

NOTE: the newly introduced get_session_source_stats() selects the
desired source (sender or receiver) by filtering them by type (using the
get_sender parameter) rather than by ssrc because this simplifies the
code and it's good enough for testing purposes as there is usually one
source per type in the test setup.

Filtering by ssrc would have required handling asynchronous signals like
"on-new-sender-ssrc", with the relative locking, just to retrieve the
actual ssrc of the sender.

rtpsource: fix stats about received packets

The update_receiver_stats() function is called also when sending packets
in rtp_source_send_rtp(), and sending packets may happen using a buffer
list rather than individual buffers.

So update the stats using the actual number of packets sent.

NOTE: this is fine for the receive path too (rtp_process_send_rtp)
because the receive path does not support buffer lists and
pinfo->packets would always be equal to 1 in this case.

test: rtpbin_buffer_list: move buffer list creation next to its validation

The tests create a buffer list and then use the chain_list callback to
verify that the correct packets have been pushed.

Move the creation and validation code next to each other so that the
reader can more easily understand what is going on.

While at it add some comments to introduce the two related functions.

test: rtpbin_buffer_list: set the chain_list function directly in the test

The helper function set_chain_function does not really do anything useful, remove it.

test: rtpbin_buffer_list: make check_packet more flexible

Make it possible to differentiate between the position in the list and
the packet index in the global structures in check_packet, in some
future case the list may change, in case some element removes a buffer
from the list, and the two indices may not coincide.

test: rtpbin_buffer_list: factor out a function to create packets buffers

test: rtpbin_buffer_list: check if the chain_list function has been called

Make the test more useful to verify that the chain list function has
actually been called.

test: rtpbin_buffer_list: port to GStreamer 1.0

Port the rtpbin_buffer_list test to GStreamer 1.0 and re-enable it.

Some other changes include:
  - the check on the caps has been moved from the buffer level to the
    pad level;
  - remove underscore prefix from static functions names, this is not
    idiomatic in C and rarely used in the other tests;
  - the unused header_buffer variable has been removed;
  - check_group() has been renamed to check_packet() because in
    GStreamer 1.0 there is no concept of "group" anymore, the comments
    have also been updated to reflect this.

tests: jpegdec: bump discoverer timeout for valgrind

Tests might take a bit longer, esp. when run under valgrind
and/or they're running on the CI with other things going on,
so let's just bump the timeout to something higher and let
the test runner time us out if needed.

meson: Only ensure that moc is available on Linux

On other OSes, it's not possible to have qmake or the qt5 pkg-config
files and not have moc, and `moc` will not be in `PATH`, so this only
causes problems.

rtpstorage: Limit the queue size

Limit to the queue size in case there is no arrival time or in case there is
a huge flood of packets.

rtpbin: Request the FEC decoder even if ignore-pt is set

rtpbin: Factor out the code that exposes the src pad

rtpreddec: Add some more debug prints

rtpstorage: Issue warning if request by size if 0

If the size is 0, then nothing will ever be in the storage, if a request is
received, it generally implies a misconfigured pipeline.

rtpstorage: Add more debug messages

rtpstorage: Make debug category available to sub objects

rtpstorage: Add debug funcptr to chain function

flac: report latency in flacenc and flacdec

The FLAC specification states that the data is processed in blocks, regardless of the number of channels. Thus, The latency can be calculated using the blocksize and rate. For example a 1 second block sampled at 44.1KHz has a blocksize of 44100

examples: rtsp: fix compiler warning

"control reaches end of non-void function"

gstrtpsession: Remove set but not use running-time

rtpmanager: Register chain functions to debug

rtpbin: Allow reusing the sender AUX bin

This is needed for the case you don't know in advance all the sessions
you will be using, but would like to place all the related AUX element
in the same GstBin. As per current implementation, each time an sender
AUX bin is requested and returned, RTPBin will walk the src pads and
create sessions for these pads.

In the current implementation, if a src pad already have a sessions, it
returns an error and stops. As a side effect, if an AUX bin is reused in
a following AUX bin request, it can only work if the pads are created on
the last request.

This change simply relax the restriction in order to keep walking, and
just ensure that all newly created pads have a sessions.

v4l2videoenc: set GstVideoCodecFrame sync point flag

The V4L2 elements already set the delta unit buffer flag when dequeueing
the buffer, but gst_video_encoder_finish_frame overwrites it from the
passed codec frame's sync point flag. Set the flag correctly.

gstrtpsession: improve stats about rtx requests

rtprtxsend: Improve looging of not found RTX packet

When an RTX packet is not found, display a message that say if the
packet have not arrived yet or if it was already removed from the RTX
packet queue.

rtpsession: Remove unused rtp_session_create_source

meson: add -Wno-unused also to C++ args when gst debug system is disabled

And check if argument is supported instead of just passing it blindly,
and make meson code slightly cleaner, centralising the argument setting
in one place.

Update LINGUAS

qtdemux: Don't pass zero to denominator for framerate

Need to respect return of gst_video_guess_framerate() to ensure
non-zero denominator.

This patch is to fix below error with an abnormal (but has valid frame) file.
(gst-play-1.0:17940): GStreamer-CRITICAL **: passed '0' as denominator for `GstFraction'

v4l2: Set Hardware classifier on encoders

v4l2: Set Hardware classifier on video decoders

v4l2transform: don't segfault if flushed without pools

The v4l2output and v4l2capture v4l2objects can have pool == NULL if they
have been stopped before.

qtdemux: Find mp4a esds atoms in protected streams sample description tables.

This problem was found in Test. 2 of the YouTube 2018 EME
tests[1]. The code was accidentally not finding an mp4a's esds atom in
the sample description table when the stream was encrypted. It assumed
that if the stream is protected, then only an enca atom will be found
here. What happens with YouTube is they often provide protected
content with a few seconds of clear content, and then switch to the
encrypted stream.

The failure case here was an incorrect codec_data field being sent
into aacparse. The advertisement of stereo audio @ 44.1kHz for the
mp4a (unprotected) stream was incorrect. As usual, the esds contained
the real values here which were mono at 22050 Hz.

Here's what the MP4 tree looks like for these types of files,
demonstrating why the code was making a wrong assumption (or maybe
YouTube is being unusual),

[ftyp] size=8+16
...
[moov] size=8+1571
...
  [trak] size=8+559
...
          [stsd] size=12+234
            entry-count = 2
            [enca] size=8+147
              channel_count = 2
              sample_size = 16
              sample_rate = 44100
              [esds] size=12+27
                ...
            ...
            [mp4a] size=8+67
              channel_count = 2
              sample_size = 16
              sample_rate = 44100
              [esds] size=12+27
                ...

In addition to fixing this, the checks for esds atoms in mp4a and mp4v
have been made symmetrical. While I haven't seen a test case for video
with the same problem, it seemed better to make the same checks. This
also fixes a crash reported from another user[2], they also noted the
asymmetry with mp4v and mp4a.

[1] https://yt-dash-mse-test.commondatastorage.googleapis.com/unit-tests/2018.html?test_type=encryptedmedia-test
[2] https://gitlab.freedesktop.org/gstreamer/gst-plugins-good/issues/398

flvmux: Fix scale of time values in warning message

rtspsrc: Don't remove udpsrc/sink from rtspsrc if they were not added to it

This can happen in various error cases that could happen between the
creation of the element in question and the adding to the rtspsrc.

It causes an ugly critical warning right now but is otherwise harmless.

test: imagefreeze: add test for the num-buffers property

imagefreeze: add a num-buffers property

The imagefreeze element can be handy for benchmarking downstream
elements because it re-uses the same buffer memory and introduces less
overhead compared to always creating new frames with videotestsrc.

However it's not possible to make imagefreeze send EOS when using
gst-launch-1.0.

Add a num-buffers property to make it look more like a source in the
above scenario.

matroskamux: add support for new color primaries

v4l2sink: fix pool-less allocation query handling

This fixes a critical warning if the last-sample property is enabled:

  (gst-launch-1.0:391): GStreamer-CRITICAL **: 01:12:57.428: gst_object_unref: assertion 'object != NULL' failed

If the allocation query does not contain any allocation pools,
gst_query_parse_nth_allocation_pool will leave the local pool,
min, and max variables undefined, so check the array length first.
If pool is NULL, do not call gst_object_unref.

meson: Build v4l2 example only if v4l2 plugin was built

Otherwise v4l2 example will be built with MSVC

docs: fix typos s/recieve/receive/

rtpsource: fix documentation of rtp_source_send_rtp parameters

In commit 28e5f9098 (rtpbin: use PacketInfo for the sender, 2013-09-13)
the rtp_source_send_rtp signature changed but the documentation was not
adjusted to match the new one.

Update the documentation to match the function signature.

rtpsession: fix typo in a comment, s/SESSION_LOCK/RTP_SESSION_LOCK/

Fix a typo in a comment, mainly to avoid confusing autocompletion in
text editors.

rtpsession: fix typos and update parameters names in comments

Some functions now accept a generic 'gpointer data' parameter because
they can work either on a single buffer or a buffer list.

However the comments were still referring to the old 'GstBuffer *buffer'
parameter, so update the comments to match the actual functions
signature.

rtpstats: fix some fields names in the RTPSourceStats documentation

Fix documentation of RTPSourceStats to use the actual fields names.

Apply "pulsesink: uncork if needed upon commit" again

This uncork condition is valid for volume ramping with silence
memchunk in pulseaudio when a stream is started.

patch history
 : 46a3c9ac8b2f8182ef8f211f4be09a67e4d42a61 - 2016-04-16 02:21:14 +1000 - Revert "pulsesink: uncork if needed upon commit"
 : 0dd46accf6d282ff07065852bd91c85c78af3394 - 2015-01-10 13:04:44 +0100 - pulsesink: uncork if needed upon commit

Change-Id: I3d9ad358f4c66df1755650960b53043df22a2d1c
Signed-off-by: Sangchul Lee <sc11.lee@samsung.com>

rtpulpfdecdec: only put recovered packet back into storage if not recovered from there

rtpulpfecdec: fix buffer leak when packet is recovered from storage

Exposed by rtpulpfecdec_recovered_from_storage test.

tests: rtpulpfec: fix buffer leak in unit test

This freed wrapped memory instead of the GstMemory or buffer.

rtph264depay: fix caps leak

Exposed by rtp_h264depay_bytestream() unit test.

tests: rtpjitterbuffer: fix leaks in new test_push_eos() test

tests: states: blacklist gtk sinks for state change test

gtk_init() throws GLib-GIO-WARNING **: unknown schema extension 'd'
unrelated to our test environment.

tests: .gitignore more test and example binaries

gtkgl: Also try retrieving an EGL context from Gdk with X11

Some embedded platforms will use EGL instead of GLX within the X11
ecosystem.

Back to development

matroskademux: fix AV1 caps when there's no codec_data

There is no "byte-stream" format for AV1 in Matroska, this
was probably cargo-culted from H.264. codec_data / CodecPrivate
is now mandatory for AV1 in Matroska[*], but there are sample
files out there which don't have it (e.g. some Elecard ones).

[*] https://github.com/Matroska-Org/matroska-specification/blob/master/codec/av1.md#codecprivate-1

meson: don't build icles when tests are disabled

They are manual tests, so let them be controlled
via the tests option.

rtpsource: small spell correct

Release 1.15.2

Update docs

Update translations

v4l2: accept Bayer as possible input/output for V4L2 codecs

A V4L2 transform codec may input/output data on Bayer format.

Add support for that.

v4l2: fix a typo on a debug message at v4l2_calls

suppored -> supported

v4l2dec: also remove the colorimetry and chroma-site fields

If a different format is chosen, then these values are incorrect.

rtpsession: Fix EOS forwarding

So far we assumed that if all sources are bye, this meant we needed to
send an EOS on the RTCP sink. The problem is that this case may happens
if we only had one internal source and it detected a collision.

So now we limit the EOS forwarding to when there is a send_rtp_sink pad
and that this pad has received EOS. We don'tcheck the recv_rtp_sink
since the code does not wait for the bye to be send before sending EOS
to the RTCP src pad.

wavparse: Declare support for RF64

RF64 encode support was added to wavenc quite some time
ago, but not declared in wavparse. It seems wavparse can
decode it though, so add it to the sink pad.

The RF64 support was added in
https://bugzilla.gnome.org/show_bug.cgi?id=735627

rtp: Add property to disable RTCP reports per internal rtpsource

This is useful when implementing custom retransmission mechanism like
RIST to prevent RTCP from being produces for the retransmitted SSRC.
This would also be used in general for various purpose when customizing
an RTP base pipeline.

rtpsession: Emit on-new-sender-ssrc for RTX ssrc also

rtp jitterbuffer test: Test for queue filling

rtpjitterbuffer: Limit size to 2^15 packets

If it goes over 2^15 packets, it will think it has rolled over
and start dropping all packets. So make sure the seqnum distance is not too big.

But let's not limit it to a number that is too small to avoid emptying it
needlessly if there is a spurious huge sequence number, let's allow at
least 10k packets in any case.

rtpjitterbuffer: There is no automatic reorder threshold

pulse: Post DEVICE_CHANGED on modification

pulse: Mark default devices as "default"

flvmux: Use 8kHz sample rate for alaw/mulaw audio

flvdemux: set sample rate to 8KHz for G.711 audio

qtmux: Only write timecode trak for video

Recent changes in ccextractor were attaching timecode meta to the closed
caption track. We shouldn't write timecode information for the closed
caption trak.

vpx: Fix build against libvpx 1.8

The deprecated debug visualizer was removed.

misc: Fix warnings on Cerbero's mingw (gcc 4.7)

error: this decimal constant is unsigned only in ISO C90 [-Werror]

pulsesink: Deal with not being able to convert a format to caps

It is possible that PulseAudio adds formats that are not yet supported
in pulsesink, and in those cases, we want to gracefully skip them rather
than cause an assert on a NULL caps.

qtdemux: Remove trailing '\n' in debug

qmlgl: Fix opengl header guard changes again

Reapply 3d708a5bfa8961cc37671bc3226976dfc9ba50ad in the correct place
after the iOS additions.

rtspsrc: use the correct segment seqnum

rtpjitterbuffer: use the correct segment seqnum

rtpsession: use the correct segment seqnum

flvdemux: Do not error out if the first added and chained pad is not linked

And let it the oportunity to get its other pad linked

Example:

```
$ gst-launch-1.0 uridecodebin uri=file:///home/thiblahute/gst-validate.save/gst-integration-testsuites/testsuites/../medias/defaults/flv/819290236.flv caps=audio/x-raw expose-all-streams=FALSE ! fakesink
Setting pipeline to PAUSED ...
Pipeline is PREROLLING ...
ERROR: from element /GstPipeline:pipeline0/GstURIDecodeBin:uridecodebin0/GstDecodeBin:decodebin0/GstFlvDemux:flvdemux0: Internal data stream error.
Additional debug info:
../subprojects/gst-plugins-good/gst/flv/gstflvdemux.c(2760): gst_flv_demux_loop (): /GstPipeline:pipeline0/GstURIDecodeBin:uridecodebin0/GstDecodeBin:decodebin0/GstFlvDemux:flvdemux0:
streaming stopped, reason not-linked (-1)
ERROR: pipeline doesn't want to preroll.
Setting pipeline to NULL ...
Freeing pipeline ...
```

webmmux: allow resolutions above 4096

Modify the caps string to allow width and height greater than 4096.
There is no need to restrict it since the matroska format allows the
width and height values to be up to eight bytes long, and this also
applies to the webm subset of the format.

Fixes https://gitlab.freedesktop.org/gstreamer/gst-plugins-good/issues/550

meson: qmlgl plugin iOS definitions

Tested with cross-ios-arm64 and cross-ios-x86, since those two are the
only archs shipped with the official Qt binaries.

qt: Don't define GLsync inside gstglfuncs.h

This was originally added for fixing conflicting definitions between
Android and Qt, but times have changed and now this breaks the build
on iOS:

 [...]/OpenGLES.framework/Headers/ES3/gl.h:1006:48: error: unknown type name 'gst_qt_GLsync'
 GL_API void           GL_APIENTRY glGetSynciv (GLsync sync, GLenum pname, GLsizei bufSize, GLsizei* length, GLint* values)   OPENGLES_DEPRECATED(ios(7.0, 12.0), tvos(9.0, 12.0));
                                               ^
 ../ext/qt/gstqtgl.h:49:16: note: expanded from macro 'GLsync'
 #define GLsync gst_qt_GLsync
                ^
 6 errors generated.

Instead, we simply avoid defining GLsync ourselves if we're using Qt.

meson: Fix indentation in qt plugin and add a FIXME comment

rtph265depay; Fix handling of marker on aggregated packet

When multiple nals are aggrgated, the marker bit should be associated only
with the last NAL of the packet. Otherwise we may break rendering in with
AU alignment.

rtph264depay: Fix handling or marker on STAP-A

Only forward the marker for the last NAL of the STAP-A. Otherwise each NAL
endup being assumed to be a full frame which may break rendering.

Fixes 557

test: h265depay: Add todo for testing aggregate packets with marker

We are missing a sample to test this, but a fix has been made, so add a
todo.

test: rtph264depay: Check handling of STAP-A marker

Related to #557

meson: orc-test is not required

This is especially never available on iOS.