Update mailing list address from sourceforge to freedesktop

gtkgstwidget: Add missing break in get_property

gtksinks: Remove undefined private structure

The classes contains a private structure which are not defined,
hence unused.

glupload: memcpy on raw data upload

Anything else requires keeping track of the GstVideoFrame mapping
across possible multiple buffers to ensure correct data pointer
usage.

glbasebuffer: add some debug and zero the data pointers on init

mpegtsmux: don't wait for data on sparse inputs like metadata or subtitle streams

mpegtsmux: add basic support for asynchronous KLV metadata streams

This is defined in SMPTE Rp 217. In this case the
metadata PES packets carry no timestamps of their
own and no Metadata Access Unit Wrappers are used.

vtdec: handle non-consecutive GstBuffer input without copying

CMBlockBuffer offers a model similar to GstBuffer, as it can
consist of multiple non-consecutive memory blocks.

Prior to this change, what we were doing was:

 1) Incorrect:

   CMBlockBufferCreateWithMemoryBlock does not copy the data,
   but we gst_buffer_unmap'd right away.

 2) Inefficient:

   If the GstBuffer consisted of non-contiguous memory blocks,
   gst_buffer_map resulted in malloc / memcpy.

With this change, we construct a CMBlockBuffer out of individual mapped
GstMemory objects. CMBlockBuffer is made to retain the GstMemory
objects (through the use of CMBlockBufferCustomBlockSource), so the
original GstBuffer can be unref'd.

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

applemedia/avsample: fix unconditional use of OSX 10.10 API

We can just not use the error/status properties

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

configure: try harder to find the right Qt5 build tools

First check for Qt build tools in the host_bins directory
from the Qt5Core pkg-config file. This fixes the situation
where both Qt4 and Qt5 are installed but the global moc/uic/etc.
are the Qt4 version, which would result in build failures.

configure/qt: Fix build without Qt5X11Extras

mpdparse: g_slice_new() can't return NULL

It will abort() if allocation fails.

mpdparser: Don't allow network access when reading the XML data

qt: don't error out if Qt X11 could not be found

PCK_CHECK_MODULES default error-if-not-found implementation errors
so override it with a simple notice that we couldn't find the library.

new qt5 qml GL video sink

Very much in the same spirit as the Gtk GL sink

Two things are provided
1. A QQuickItem subclass that renders out RGBA filled GstGLMemory
   buffers that is instantiated from qml.
2. A sink element that will push buffers into (1)

To use
1. Declare the GstGLVideoItem in qml with an appropriate
   objectName property set.
2. Get the aforementioned GstGLVideoItem from qml using something like

QQmlApplicationEngine engine;
engine.load(QUrl(QStringLiteral("qrc:/main.qml")));

QObject *rootObject = engine.rootObjects().first();
QQuickItem *videoItem = rootObject->findChild<QQuickItem *> ("videoItem");

3. Set the videoItem on the sink

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

bluez: Fix make dist while bluez >= 5 is not available

mpegtsbase: Fix previous commit

We only want to do a hard reset of the observations if we're working
with TIME segments in push mode. For BYTE segment we want to keep
the observations (in order to do seeks in push-mode).

mpegtsbase: Flush more on DISCONT buffers in push mode

When in push mode, we want to discard all previous observations from the
mpegtspacketizer when we get a DISCONT buffer.

This avoids trying to calculate bogus timestamps (estimating them using old
PCR observations).

We only do a hard reset in push-mode. In pull-mode we still need the observations
(in order to seek properly)

gitignore: Add new generated files

bluez: Include new file in dist'ed files

Fixes make distcheck and tarball generation

gtk: add to the generic/states test

GstGtkGLSink: Ensure widget has a toplevel parent

Checking for a parent is not enough, it must have a toplevel one.
If widget has no toplevel parent then add it in a GtkWindow, that
make it usable from gst-launch-1.0.

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

GstGtkGLSink: Post error if widget gets destroyed

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

GstGtkGLSink: fix possible warning in finalize

If the element is finalized before going in READY state
the widget could still be NULL.

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

mpdparser: Fix some memory leaks in the MPD parser and unit test

dashdemux: tests: added unit tests to test all functions

Added unit tests for all functions. Code coverage:
Overall coverage rate:
  lines......: 83.8% (1941 of 2316 lines)
  functions..: 100.0% (141 of 141 functions)

pngparse: fix a GstCaps object leak

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

tsdemux: remove unnecessary check

This is not public API, use g_assert() instead of
g_return_if_fail(), so that it's compiled out in
releases. It's only called from our code, with &foo.

tsdemux: add support for KLV metadata streams

There's no timestamps for these streams though, we
might want to make some up based on the last/next
video PTS or so.

Revert "Revert "dashdemux: fixed gst_mpd_client_advance_segment to return GST_FLOW_EOS""

This reverts commit 4875ddf5855a26f349df7b385b68eb692e314bfa.

This was based on a misunderstanding of the code.

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

gl/eagl: Don't call anything synchronously from the main thread

This will deadlock if the main thread is the one who creates the GstGLContext.
All things we call from the main thread should be possible from any thread.

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

bluez: remove unnecessary goto

All goto fail happen before ret is set. ret must be NULL, and the only
thing the fail statement block does is return NULL. Replacing the jumps to
do this return directly.

CID #1311329

dashdemux: remove redundant else statement

Else statement is redundant when the if block finishes with an return.

androidmedia: Add support for H265/HEVC

vtdec: fix refcount error in error handling

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

adaptivedemux: Explicitly check for CUSTOM_ERROR when converting errors to EOS

We use things like CUSTOM_SUCCESS* in other places, and those are not errors
but should just be handled like OK.

Revert "dashdemux: fixed gst_mpd_client_advance_segment to return GST_FLOW_EOS"

This reverts commit dfe37ffc59fdf5f4075696976d98ec8d6db73138.

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

mpdparser: Fix off-by-one in has-next-segment calculation

adaptivedemux: If downloading the last fragment fails, consider this as EOS and not ERROR

Sometimes the last fragment does not exist because of rounding errors with the
durations. Just finish the stream gracefully instead of erroring out instead.

download: only start a download transfer for sysmem caps features

dashdemux: tests: added indentation in xml strings to improve readability

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

dashdemux: added parsing of bitstreamSwitching@AdaptationSet

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

dashdemux: regrouped functions in gstmpdparser.h file

Moved gst_mpd_client_get_next_segment_availability_end_time and
gst_mpd_client_add_time_difference functions to be grouped with
functions from the same category.

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

dashdemux: tests: corrected code layout

deleted the empty line between gst_mpd_parse call and its check so that
the assert is next to the function it is testing

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

mpegvideometa: add meta transform function

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

glsyncmeta: transform func: return FALSE if not supported or failed

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

dashdemux: fixed gst_mpd_client_advance_segment to return GST_FLOW_EOS

Fixed gst_mpd_client_advance_segment to return GST_FLOW_EOS if the
new index is out of range.

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

android: add missing egl.h include

gltransformation: correct vao usage

keep the vao bound after uploading the new vertex data

fixes a mesa GL error "no vertex array object bound" on caps changes

gl: consolidate egl header includes to egl-only headers

They may conflict with other headers.

gtkglsink: Release the widget lock when trying to get the GL context

Otherwise we might be waiting for the lock on the main loop (for
example in the ->render vmethod) and thus we will deadlock.

videoaggregator: Remove pixel-aspect-ratio field from the caps returned by getcaps

Avoiding not negotiated error while negotiating as we anyway force 1/1
as output

curlsshsink: remove redundant else statement

Else statement is redundant when the if block finishes with an return.

dashdemux: fixed segment start when specified by SegmentTimeline

@t attribute in segment Timeline is relative to the period start.
Corrected the code to take this into account when computing segment
timestamp.

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

dashdemux: wrong segment duration computed by gst_mpd_client_setup_representation

Corrected computation of segment start and duration.

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

dashdemux: corrected search for audio languages

Corrected the initialisation of mimeType in
gst_mpdparser_get_list_and_nb_of_audio_language: the variable is used
in a loop, so it must be set to NULL at the beginning of each iteration.

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

Automatic update of common submodule

From f74b2df to 9aed1d7

glvideomixer, glmixer: Add description and klass

build: Fix broken BlueZ configure check

The previous commit left an extra set of brackets, and missed removing
the explicit dbus check (which is no longer needed since we're using
gdbus).

build: remove support for bluez4

bluez: refactor to use glib and add connection state tracking

pcapparse: fix another regression

Introduced by c4c9fe60b pcapparse: Take buffer directly from the adapter

Using gst_adapter_take_buffer_fast() can lead to buffers that are
made up of multiple memories with the first memory smaller than the
RTP header size, which violates assumptions GstRtpBaseDepayloader
makes, namely that the complete RTP header will be in the first
memory. This leads to such packets being dropped when feeding
them from pcapparse to RTP depayloaders. Use take_buffer() so
we get buffers with a single memory.

tests: pcapparse: add unit test for frames with eth padding

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

pcapparse: fix regression when handling packets with eth padding

Introduced by c4c9fe60b pcapparse: Take buffer directly from the adapter

Flush any trailing bytes after the payload from the adapter as well,
otherwise we'll read a bogus packet size from the adapter next and
then everything goes downhill from there.

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

openh264dec: Unref video frames and codec state in all cases

gl: initialize output params to 0 before calling gl functions

The client side API of the Chromium's GPU Process has asserts
in debug mode that check that output params are initialized to 0.

gl: Don't leak pool if set_config failed

glfilter: Don't cache buffer pool

Caching and sharing to multiple element the same pool prevents
renegotiation from passthrough to not passthrough.

openh264dec: Uninitializing the decoder can't fail and will always return 0 == cmResultSuccess

... but when we cast cmResultSuccess to a gboolean it will be FALSE, and make
GstVideoDecoder::stop() fail for no reason.

glimagesink: use g_clear_error instead of g_error_free

replace g_error_free with g_clear_error, as it internally
checks if error variable is valid or not.

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

amc: fix check whether we managed to acquire a buffer

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

srtpdec: Add config for the replay window size

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

dashdemux: corrected parsing of %u in segment templates

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

dashdemux: corrected recognition of $$$ in segment templates

fixed recognition of $$ after a $ that ends an identifier, eg $Time$$$

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

androidmedia: Call initialize_classes() also when we got the Java VM from the app

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

gl: add missing break

gst_gl_view_convert_element_set_property() is missing a break at the end
of the PROP_OUTPUT_DOWNMIX_MODE case. Adding it.

CID #1308949

glimagesink: Don't leak pool

gst_query_add_allocation_pool is transfer none. Also unref
if there was a configuration error.

mpegdemux: check pointer before dereferencing

gst_ps_demux_get_stream() could return NULL when it is unknown stream,
check this hasn't happened before dereferencing the returned pointer.

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

dashdemux: corrected next fragment duration validation

Before returning the next fragment duration value, the
gst_mpd_client_get_next_fragment_duration function tries to validate it.
But the condition was incorrect.

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

adaptivedemux: Move unref() into the right scope

adaptivedemux: Properly handle presentationTimeOffset for seeking and multi-period streams

Segment start/time/position/base should only be modified if this is the first
time we send a segment, otherwise we will override values from the seek
segment if new streams have to be exposed as part of the seek.

Segment base should be calculated from the segment start based on the stream's
own segment, not the demuxer's segment. Both might differ slightly because of
the presentationTimeOffset.

Always add the presentationTimeOffset (relative to the period start, not
timestamp 0) to the segment start after resetting the stream's segment based
on the demuxer's segment (i.e. after seeks or stream restart). Also make sure
to keep the stream's segment up to date and not just send a new segment event
without storing the segment in the stream.

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

dashdemux: Subtract the period start time from the presentation offset

We're interested in the offset between the period start timestamp and the
actual media timestamp so that we can properly correct for it. The absolute
presentation offset to timestamp 0 is useless as the only thing we really
care about is the offset between the current fragment timestamp and the
media timestamp.

dashdemux: Subtract the period start when seeking based on a template

Otherwise we will look for segments after the period usually. The seek
timestamp is relative to the start of the first period and we have to
select a segment relative to the current period's start.

dashdemux: Include the period start in the fragment timestamps in all cases

We didn't do this for fragments that are generated on demand from a template,
only for the other cases when they were all generated upfront. This caused
fragment timestamps to start from 0 again for each new period.

dashdemux: Seek on the new streams if the seek caused a period switch

Seeking on the old streams is pointless, they are going to be freed on the
next opportunity.

dash: Correct check for valid timeShiftBufferDepth

If not set, the timeShiftBufferDepth has a default value of -1.
The standard says that this should be interpreted as infinite.

The gst_mpd_client_check_time_position function incorrectly compares
timeShiftBufferDepth with 0 instead of -1 to determine if it was set.

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

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