docs: decodebin2 -> decodebin

applemedia: enable sharing of CMBlockBuffer data

Instead of wrapping with GST_MEMORY_FLAG_NO_SHARE, we make the GstMemory
object retain the underlying CMBlockBuffer.

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

applemedia: CMBlockBuffer can be non-contiguous

CMBlockBufferGetDataLength would return the entire data length, while
size of individual blocks can be smaller. Iterate over the block buffer
and add the individual (possibly non-contiguous) memory blocks.

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

dashdemux: Corrected duration argument for gst_mpd_client_add_media_segment call

The last parameter of gst_mpd_client_add_media_segment function is a
duration. But when called from gst_mpd_client_setup_representation, the
last argument was wrongly set to PeriodEnd

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

dashdemux: incorrect use of period start

The period start information, calculated in gst_mpd_client_setup_media_presentation
function is stored in stream_period->start. The information read from
xml file and stored in stream_period->period->start is not changed.

If the xml file does not contain the period start information,
stream_period->period->start will be -1.

The function gst_mpd_client_get_next_segment_availability_end_time wants to
use period start time, but incorrectly uses stream_period->period->start
(value from xml file, which could be -1) instead of stream_period->start
(computed value)

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

tests: dashdemux: renamed functions

Renamed 2 testcase functions:
- dash_mpdparser_program_information
- dash_mpdparser_base_URL

Back to development

Release 1.5.2

Update .po files

tests: dashdemux: added explicit type conversions

flexelint (http://www.gimpel.com/html/flex.htm) static code analyser
complained about implicit conversions from unsigned to signed, so I added
explicit conversions.

Ideally, the size parameter of gst_mpd_parse function should be unsigned,
but I don't want to change the API.

tests: dashdemux: corrected return type for duration_to_ms function

The duration_to_ms function converts a time specified by year, month, day,
hour, minute, second, millisecond to a millisecond value. Because all the
arguments are positive numbers, the result must also be positive.

This patch changes the returned value from a gint64 to a guint64 type.

po: Update translations

adaptivedemux: Also update segment.base and segment.time when switching periods

And include the presentation offset in the last known position for each
stream, and just because we can also keep track of the latest known position
inside the demuxer segment.

Revert "gl/window/x11: don't create our own X11 display"

This reverts commit 5697b6b89b4b2a15c45bd47be940a17f4412ea11.

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

Revert "eagl: Don't dispatch_sync() to the main thread if we are on the main thread"

This reverts commit 0bff481011cf74c59869b511393d1696b570e3d5.

It wasn't supposed to be merged and also doesn't fix the problem.

adaptivedemux: Remember the last return value of gst_adaptive_demux_stream_update_fragment_info()

It's going to return EOS if the period ended or otherwise there is just no
next fragment left. If we don't store the last return value, it will always
stay OK and gst_adaptive_demux_combine_flows() will always return OK instead
of EOS once all streams are done.

This partially switches period changes in DASH by at least trying to switch
instead of just stopping. What is still left is that after a period change
with DASH the times all start at 0 again instead of continuing.

eagl: Don't dispatch_sync() to the main thread if we are on the main thread

This will otherwise deadlock.

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

gl: Include texParameterf in the GL function table

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

error-ignore: New element to convert some GstFlowReturn types into others

Can be used to fix misbehaving sinks. It will pass through all buffers
until it encounters GST_FLOW_ERROR or GST_FLOW_NOT_NEGOTIATED (configurable).
At that point it will unref the buffers and return GST_FLOW_NOT_LINKED
(configurable) - until the next READY_TO_PAUSED or FLUSH_STOP.

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

videoaggregator: simplifies and improves sink_get_caps.

The problem here was that after removing the formats and
all the things we could convert, we then intersected these
caps with the template caps.

Hence if a subclass offered permissive sink templates
(eg all the possible formats videoconvert handles), but only
one output format, then at negotiation time getcaps returned
caps with the format restricted to that format, even though
we do handle conversion.

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

motioncells: Port to g_get_current_time

This is more portable.

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

avfvideosrc: add frame rate range to caps

When AVFoundation indicates a supported frame rate range, add it to
the caps. This is important for devices such as the iPhone 6, which
indicate a single AVFrameRateRange of 2fps - 60fps.

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

tests: dashdemux: added parsing of period element

Improved dash_mpd unit tests by adding new tests that parse the Period element.

Code coverage reported by lcov for dash/gstmpdparser.c is:
  lines......: 43.0% (985 of 2290 lines)
  functions..: 47.5% (67 of 141 functions)

glmemory: fix consistency about pbo availability.

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

tests: dashdemux: extended MPD element test to test xml namespaces

Extended the dash_mpdparser_mpd testcase to also test parsing the xml
namespace attribute.

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

opusenc: Add bitrate to the tags

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

dashdemux: corrected computation of period's duration

According to ISO/IEC 23009-1:2014(E), chapter 5.3.2.1
"The Period extends until the PeriodStart of the next Period, or until
the end of the Media Presentation in the case of the last Period."

This means that a configured value for optional attribute period duration
should be ignored if the next period contains a start attribute or it is
the last period and the MPD contains a mediaPresentationDuration attribute.

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

bayer: update ORC files

Update ORC files with newest ORC to fix backup code generation error.

irtspparse: reflow code to avoid uninitialized variable warning

This should hopefully allow even the most challenged static
code analyzer to figure out that it's all fine. Also makes
the flow clearer.

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

tsmux: code style fix

trivial patch to add proper ( while checking for if(G_UNLIKELY())

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

uvch264src: fix small leak in caps query handler

gst_query_set_caps_result() does not take ownership of the caps.

uvch264_src: add missing break in caps query handler

There is a missing break statement in switch,
which will result in executing default case as well
resulting in wrong behavior

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

mpeg1sys: remove ancient plugin that's never even been ported to 0.8

codecparsers: jpeg: add some padding to ScanHdr struct

For bits that we don't parse out at the moment.

codecparsers: jpeg: add new API to .def file

codecparsers: jpeg: fix docs for table parsing functions

docs: add new JPEG codecparser API to the docs

And sprinkle some more Since markers

examples: move vp8 parser test to codecparser example directory

examples: add small jpeg codecparser test

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

codecparsers: jpeg: fix validity checking of data parsed

g_return_val_if_fail() and g_assert() are not appropriate
for checking untrusted external data.

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

codecparsers: jpeg: fix up API

- add data pointer to GstJpegSegment and pass segment
  to all parsing functions, rename accordingly
- shorten GstJpegMarkerCode enum type name to GstJpegMarker
- move function gtk-doc blurbs into .c file
- add since markers
- flesh out docs for SOF markers

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

codecparsers: jpeg: tweak API a little

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

codecparsers: jpeg: hide gst_jpeg_scan_for_marker_code()

Make this function private for now, since it's unclear whether
it's actually needed seeing that gst_jpeg_parse() scans too.

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

codecparsers: jpeg: fix and optimize scan for next marker code.

Fix scan for next marker code when there is an odd number of filler
(0xff) bytes before the actual marker code. Also optimize the loop
to execute with fewer instructions (~10%).

This fixes parsing for Spectralfan.mov.

codecparsers: jpeg: fix calculation of segment size.

The size of a marker segment is defined to be exclusive of any initial
marker code. So, fix the size for SOI, EOI and APPn segments but also
the size of any possible segment that is usually "reserved" or not
explicitly defined.

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

codecparsers: jpeg: fix default Huffman tables generation.

Fix build_huffman_table() to correctly fill in the associated HUFFVAL
entries to the default Huffman tables.

codecparsers: jpeg: add JPEG bitstream parser

Signed-off-by: Gwenole Beauchesne <gwenole.beauchesne@intel.com>
https://bugzilla.gnome.org/show_bug.cgi?id=673925

h264parse: fix typo in log message

gldisplay: add gst_gl_display_create_context

It also emits a create-context signal so that an application
can provide an external GstGLContext backend.

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

glcontext: move display from priv

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

Revert "glcontext: add gst_gl_context_set_display helper"

This reverts commit 71b8103cbd16fff9cf5a65cf517083cb794aa3b5.

Revert "gl: add GstGLContextGPUProcess backend"

This reverts commit b377112ee38912d316e77b4e2102041389dc0051.

glimagesink: Remove duplicate, useless line of code

opengl: glstero* are only built with full OpenGL

Don't try to register the elements unless they are built.

style: Remove white spaces at the end of line

glviewconvert: Fix GLES2 compatibility

glviewconvert: Fix broken validity check

h264parse: Move PAR calcs, and use them for stereoscopic half-aspect

Move the pixel-aspect-ratio calculations higher up in caps
determination, so the results are available for a call to
gst_video_multiview_guess_half_aspect() when stereoscopic video
is detected.

3dvideo: Add simple gtk example stereoscopic video player

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

gl: Add glviewconvert, glstereomix and glstereosplit elements

Conversion elements for transforming multiview/stereoscopic video

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

glimagesink: Support multiview/stereoscopic video

Support video with multiview info in the caps, transform
it to mono anaglyph by default, but allow for configuring
other output modes and handoff to the app via
the draw signal.

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

gl libs: Add glviewconvert helper object

Add API for a helper object that can convert between different
stereoscopic video representations, and later do filtering
of multiple view streams.

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

dashdemux: added warnings in gst_mpd_client_setup_streaming function

Added some warning messages in gst_mpd_client_setup_streaming to help
debug situations when the function will return FALSE.

Renamed a wrongly spelled variable.

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

dashdemux: corrected comments in gstmpdparser.h

Corrected some comments in gstmpdparser.h file.
Moved gst_mpd_client_get_adaptation_sets function to be grouped with
other functions from AdaptationSet group

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

dashdemux: fixed getting representation based on max bandwidth

The gst_mpdparser_get_rep_idx_with_max_bandwidth function assumes
representations are ordered by bandwidth and incorrectly returns the
first one when wanting the one with minimum bandwidth.

Corrected gst_mpdparser_get_rep_idx_with_max_bandwidth function to get the
correct representation in case max_bandwidth parameter is 0.

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

androidmedia: Allow the application to provide the Java VM

In JNI_OnLoad() we will already get the Java VM passed and could
just directly use that. gstreamer_android-1.0.c will now provide
this to us.

Reason for this is that apparently not all Android system are
providing the JNI functions to get the currently running Java VMs, so
we would fail to get. With this we will always be able to get the Java
VM on such systems.

androidmedia: Prefer software codecs over hardware codecs for audio

Hardware codecs don't make sense for audio and are generally less reliable on
Android than the software codecs.

androidmedia: Failing to detach a thread is not that much of a problem

Someone else might have detached it before us, so make this just normal debug
output instead of a GST_ERROR()

androidmedia: Don't fail if JNI_CreateJavaVM can't be found

We only need that if no Java VM is running yet, and all usual cases,
i.e. when calling GStreamer from an actual Android app, there will already
be a Java VM we can just use.

It seems like some phones come without that symbol, let's hope they come
with the other symbol but for now don't make a missing JNI_CreateJavaVM fatal.

fpsdisplaysink: get rendered and dropped stats from QOS messages

Use QOS messages to update rendered and dropped frame stats. This is
the only accurate method. The old method didn't take max-lateness and
latency into account.

nalutils: trivial patch to check if

trivial patch to add proper ( while checking for if(G_UNLIKELY())

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

Automatic update of common submodule

From 6015d26 to f74b2df

gtkglwidget: Const'ify another array

gtkglwidget: Calculate the viewport size ourselves

Getting the current viewport and modifying it relatively will produce an
interesting feedback loop during widget resizing. Over a few frames we
will gradually move the viewport a bit until it converged again, adding
unnecessary additional borders at the top and left.

gtk: Use the display width/height for the widget's preferred width/height

gtksink: Add support for xRGB/BGRx

gtk: Cairo color formats are in native endianness, GStreamer's in memory order

CAIRO_FORMAT_ARGB32 is ARGB on big endian and BGRA on little endian.

gtk: Implement ignore-alpha property and enable it by default

gtk: Sync properties from the sink to the widget upon widget creation

mpegtsmux: Remove redundant min_dts

After few iteration, this variable became the same as dts. It's not
the min as the name says, but the dts of the current buffer. Simply
remove and place with dts. Also move the debug trace to actually
print the signed version of the running-time dts.

glimagesink: add missing handle-events/ignore-alpha property to the bin

gtk: implement pixel and display aspect ratio handling

videoaggregator: Print some debug output if we change the timestamp offset

videoaggregator: Add some more debug output

androidmedia: Move variables into the scope where they are needed

videoaggregator: Don't update the ts-offset before updating the actual configured caps

videoaggregator: Remember if the last LATENCY query returned live or not and use the in the QoS messages

androidmedia: Don't query supported color formats for non-video codecs

mpegtsmux: fix build error

after e000a6f0a4984fbe81190df6cd439c70dafde3d6, there is build error in bad plugins
this happens because, GST_CLOCK_STIME_IS_VALID () is being checked for pad_data
but it expects a GstClockTime parameter. Changing the check to 'dts'

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

gl/examples: update qt examples for api changes

gtk: silence unused variable warnings for unsupported winsys'

gtk: implement basic wayland GL support

compositor: update zorder documentation.

It is not bound between 0 and 10000 anymore.

videoaggregator: No need to artificially bound the zorder.

It is an unsigned integer so the upper bound is G_MAXUINT.

codecparsers: mpeg4: actually return full number of bits of resync marker

Switch the increment of markersize from when it is used to when it is
returned from compute_resync_marker_size.

This also makes the CHECK_REMAINING in gst_mpeg4_parse_video_packet_header
check for the actually required number of bits now and not one too few.

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

Revert "codecparsers: remove ignored increment of return"

This reverts commit 916b954315abc2f94348ec0be3e116c19b080b54.

Clearly something else was intended, and it also makes
more sense to add the extra bit. The resync marker is
N zero bits plus a 1 bit, and the pattern/mask needs to
be run on N+1 bits too.

(Even after the rever the code doesn't do that of course, so
it still needs to be fixed differently.)

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

opensles: Fix build with Android API level < 14

Headers were broken on older Android versions, apparently.

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

openslessink: Allow setting the stream type via a property

openslessrc: Implement recording presets

This allows us to signal what kind of audio we are expecting to record,
which should tell the system to apply filters (such as echo
cancellation, noise suppression, etc.) if required.

decklinkvideosink: Remove late frame-dropping workaround for basesink bug

This was fixed by https://bugzilla.gnome.org/show_bug.cgi?id=749258
in basesink, and is not necessary to duplicate here anymore.