gstreamer: bad: Fix memory leaks when context parse fails

When g_option_context_parse fails, context and error variables are not getting free'd
which results in memory leaks. Free'ing the same.

And replacing g_error_free with g_clear_error, which checks if the error being passed
is not NULL and sets the variable to NULL on free'ing.

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

assrender: optimize blending

Skip empty source pixels when blending an image. There can be hundreds
of images to blend at each frame, so this brings a nice performance
boost.

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

assrender: Use the window size from downstream

This allows rendering the overlay at a resolution matching the video
resolution.

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

assrender: Improve negotiation

This is mostly a copy/paste of the negotiation function in
basetextoverlay, which was improved recently to handle many more cases.

This will allow us to negotiate a window size with downstream.

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

assrender: render the subtitles with the proper aspect

This bumps the libass version requirement to 0.10.2.

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

dashdemux: test: added unit test for presentation time offset

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

dtls: require openssl >= 1.0.1

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

dtls: fix printf format on win32

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

dashdemux: Correctly assign earliest pts instead of overriding it with first offset

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

qt: add support for building on osx/ios

Including:
- Necessary configure checks
- Necessary compile time platform checks
- Necessary runtime qt iOS/OSX platform detection

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

rtmpsrc: Fix indentation with gst-indent

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

audiovisualizers: Fix nodist_HEADERS

Remove baseaudiovisualizer test.

The base class has moved into gst-plugins-base

aggregator: Convert GST_ERROR_OBJECT() for seek events to GST_DEBUG_OBJECT()

aggregator: For the start time selection, only set the segment position

segment.time and segment.start can stay the same, and were always the same
before anyway because of a mistake.

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

audioaggregator: Select the initial offset based on the start segment position

instead of always using 0. Otherwise we might output a lot of silence in the
beginning instead of outputting from the relevant position.

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

audiovisualizers: merge audiovisualizer base classes

These plugins now use the audiovisualizer base class in pbutils

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

gl: move debugging related functions to their own file

glcolorscale: use glbasefilter vfuncs

Uses less code \o/
Fixes legacy opengl rendering \o/

mssdemux: activate streams before configuring bitrate

Doing the contrary has no effect and the consequence is that playback
will start with the lowest bitrate even if we can already handle
higher bitrate.

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

mssdemux: PlayReady WRM parsing support

If the manifest has a ProtectionHeader node then parse it and emit
protection events according to the specified protection SystemID.

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

mssdemux: always use timescale for gst timestamp calculation

Not doing this can lead the demuxer to attempt downloading fragments
for an invalid start time. The server would then send a HTTP
Precondition failed error, the demuxer would try some more times to
download the invalid fragment and eventually error out.

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

gl: Pass GL_CFLAGS to g-ir-scanner

This unbreaks building when some headers are under a non-standard path.
e.g. /usr/X11R6/include as on OpenBSD.

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

applemedia: dist missing header files

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

audiointerleave: typecast bit-mask to guint64 to fix segmentation fault

While creating caps in audiointerleave tests, bitmask is being set as 0x9
This is resulting in segmentation fault. Fix the same by typecasting to guint64

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

glcontext: fixup strstr lengths so we don't overrun

glupload: remove useless release_buffer

It's a leftover from when we weren't outputting GstBuffer's and
returning raw texture id's.

glutils: use gst_element_set_context for setting display/other_context

1. So we get tracking inside GstElement properly when e.g. adding to a bin
2. Removes redundant code.  Now only one place where
   GstContext->GstGLDisplay/GstGLContext transformation occurs
3. Fixes a memory leak in the process
4. Make the retrieval of debug categories thread safe

gtk: add some GL debug statements to show up in GL traces

hls: m3u8playlist: more clean-ups

Remove unused functions; move GstM3U8 struct into .c file,
and indent the header so it's nice to read.

hls: m3u8playlist: fix silly queue iteration code

hls: m3u8playlist: more minor playlist_render() clean-ups

Move the TAG defines directly into the code, not sure what
their purposes is, these are printf format strings so having
them directly as literals in the code where they're used
makes the code easier to follow.

hls: m3u8playlist: move entry rendering into render_playlist()

Makes the code easier to follow and avoids unnecessary temporary
strings, since we can just append to the playlist GString directly.

hls: m3u8playlist: simplify gst_m3u8_playlist_render()

Remove playlist_str GString variable from GstM3U8Playlist struct,
since it's only used temporarily in playlist_render(). Might just
as well keep it local then.

hls: remove unused macro and direct gio usage

mpegtsmux: fix downstream key unit events handling with hlssink

The buffer timestamps in the collect function will already be
running time, don't try to convert them again to running time,
this would yield CLOCK_TIME_NONE now that the segment is shifted
to account for negative dts.

This fixes x264enc ! mpegtsmux ! hlssink, which was broken
because mpegtsmux would send a downstream key unit event with
running time NONE and then hlssink would immediately send
another one upstream and it would just be a flood of force
keyframe events in both directions after the first one. This
would then break hlssink because it uses multifilesink in
next-file=key-unit-event mode, and starting a new file after
every few kB does not work well for HLS.

opencv: erode: remove non-ip transform func

Transform is set to be done in place in gstcvdilateerode.c, so the in-place
transform function is always used and the other is redundant. Removing it.

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

opencv: dilate: remove non-ip transform func

Transform is set to be done in place in gstcvdilateerode.c, so the in-place
transform function is always used and the other is redundant. Removing it.

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

opencv: use gst_opencv_video_filter_set_in_place()

Using the gst_base_transform function directly won't work. Need to use
gst_opencv_video_filter_set_in_place().

handdetect: check for haar cascade files path

When running GStreamer from uninstalled sources, the location of the haar
cascade files will be local. Check if running in uninstalled and set the
file paths accordingly.

adaptivedemux: remove unused variable

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

videoaggregator: fix compilation with older glib version

Remove weird use of private gtype defines and fix compilation
with older glib versions such as 2.36.

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

opencv: cvlaplace: add scale and shift properties

Add scale and shift properties so these can be used to tweak the values
used in cvConvertScale() instead of using only defaults.

qtsink: explicitely fallthrough switch statement

In case ret is False, fallthrough to default case.

CID #1320705

vc1parse: add break to switch case

Even though all cases inside VC1_STREAM_FORMAT_ASF are goto or
g_assert_not_reached(), add a break at the end to appease Coverity.

CID #1320706

assrender: fix leak of tag samples in the tag list

Move handling of a GstSample in a separate function, and unref the
sample after calling it. libass copies the font data so we don't need to
keep it around.

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

assrender: fix gap event leak and invalid return value

We don't want the gap event to be forwarded

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

nvenc: call the parent class on GstElement::set_context

gl/eagl: use the default GL context debug category

Add Nvidia based hardware encoder element

Currently only h264 is supported

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

gtk: fix assertion when the element has no peer

When proxying keyboard/navigation/mouse events, only unref a successfully
retreived peer pad.

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

Revert "midi: add an ALSA MIDI sequencer source"

This reverts commit cd7f4e524718a2f75467852235e4e13d64d2ac82.

gl: set the context on the element on a context query

Otherwise it's possible to lose the context information if the
context is only propagated through queries.

gl: chain up to the parent class for GstElement::set_context

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

midi: add an ALSA MIDI sequencer source

The alsamidisrc element allows to get input event from ALSA MIDI
sequencer devices, and possibly convert them to sound using some
downstream element like fluiddec.

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

qml: remove overwritten value

Value in tex is overwritten before being used. Removing it.

CID 1320715

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

qt: add support for building/running on android

Including:
- Necessary configure checks
- Necessary compile time platform checks
- Necessary runtime qt android platform detection
- Escaping GLsync definition with Qt's GLES2 implementation

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

qt: don't use CPPFLAGS for tools that cannot use them

For example moc will bail out when given arguments it does not
know about.  The moc specific MOC_CPPFLAGS can still be used
to pass flags to moc.

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

qt: rename library to include gst prefix

libqtsink -> libgstqtsink

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

gl: sprinkle some debug markers to ease debugging

gl/utils: add a function to insert a debug marker

These markers are visible in tools that record the GL function calls
such as apitrace, et al.

Makes it easier to match up GL draw commands with specific elements.

gl: add some debugging prototypes

glwindow: remove unused draw_unlocked function

The functionality is provided by draw anyway and is leftover from
X11's specific threading requirements that no longer apply.

resindvd: Send gap updates to the segment stop during stills

Ignore the normal gap threshold for laggy streams and
immediately catch all streams up to the end of the segment
when processing gap updates for a segment during a
still frame sequence.

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

Revert "dvdspu: render to AYUV overlay"

This reverts commit 5016a73190595505dc38b54f8f9a08c4f180f3a6.

Revert "dvdspu: render to ARGB overlay instead of AYUV"

This reverts commit dd3e9deb2aa695a391b58f24d86a3c00bbc1258a.

Revert "dvdspu: handle frame size event from upstream"

This reverts commit 46aaaa6c309b8a8f6e218142fd5944f5d24f095c.

Revert "dvdspu: cache overlay composition"

This reverts commit aabb8a1a68372f750c63b9b6586e18e829745774.

Revert "dvdspu: improve negotiation of overlay composition"

This reverts commit 1899e2a099932a7a734e7784c20b624517237e18.

dvdspu: fix pgs palette colors

U and V were inverted

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

dvdspu: improve negotiation of overlay composition

Support negotiating GstVideoOverlayComposition downstream
while not providing it upstream.

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

dvdspu: cache overlay composition

This avoids rendering the overlay buffer for each video frame.

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

dvdspu: handle frame size event from upstream

An IDX file or codec_data normally contains the original frame size of
the video. Allow upstream to provide this information by sending a
custom event, which will allow scaling the overlay correctly.

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

dvdspu: allow suffix in dvd event name to allow multiple sticky dvd events

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

dvdspu: render to ARGB overlay instead of AYUV

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

dvdspu: render to AYUV overlay

Instead of only supporting writing SPU data directly to YUV frames,
render the SPU data to an intermediate AYUV overlay buffer. The overlay
data is then attached to the video frame if downstream supports overlay
composition, otherwise the AYUV overlay is blended to the video frame.

For the PGS format, the overlay buffer size is set to the size of the
Composition Window, and its position in the overlay composition is set
to the window position. The objects to render are now cropped when the
cropping flag is set.

For the Vobsub format, the overlay buffer size is set to the size of the
Display Area.

Once rendered, the overlay composition rectangle is now moved and scaled
to fit the video output size, to avoid clipping.

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

dvdspu: pass dvdspu argument to set_caps functions

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

dvdspu: skip unneeded reading of RLE data

The RLE data was being read for dumping the SPU image even when the
DUMP_FULL_IMAGE macro was not set.

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

check: Add test for videoaggregator sinkpads being sorted by zorder

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

audiomixer: fix deadlock when G_DISABLE_ASSERT is not defined

This makes the audiomixer unit test time out in master.
Broke with 587e7c4

dash: Fix unit test after internal API change

sfdec: Fix typo in goto variable name

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

dashdemux: create src pads for subtitle streams.

Create src pads for Representations that contain timed-text subtitles,
both when the subtitles are encapsulated in ISO BMFF (i.e., the
Representation has mimeType "application/mp4") and when they are
unencapsulated (i.e., the Representation has mimeType
"application/ttml+xml").

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

audioaggregator: Stop using deprecated gst_segment_to_position()

aggregator: Don't forward QOS events to sinkpads that had no buffer yet

Otherwise they will receive a QOS event that has earliest_time=0 (because we
can't have negative timestamps), and consider their buffer as too late

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

dashdemux: Implement lazy-loading of external periods

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

mpdparser: Load OnLoad external resources immediately instead of on demand

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

mpdparser: Add support for loading external SegmentLists

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

mpdparser: Add support for loading external AdaptationSets

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

mpdparser: Implement loading of external Period nodes

The same has to be done for AdaptationSet and SegmentList nodes still.

Also this does not correctly implement the semantics: by default Period (and
other nodes) should only be loaded when needed, not in the very beginning. We
need to implement lazy loading for them, which means adjusting
gst_mpd_client_setup_media_presentation().

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

mpdparser: Store an URI downloader in the parser for downloading additional MPD resources if needed

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

mpdparser: Parse xlink attributes from Period, AdaptationSet and SegmentList

We still have to do something useful with them, like actually loading the
content behind the URL.

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

dashdemux: Properly handle relative and non-HTTP URIs for the headers/indices

gst_uri_join_strings() will return the second parameter if it is an absolute
URI. No need to do a (wrong) check if the URI is absolute or not beforehand.

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

mpdparser: If no Initialization is present in the SegmentBase, assume it is before the indexRange

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

Back to development

Release 1.6.0

Update .po files

glfilter: use GL_ELEMENT_ARRAY_BUFFER for vbo indices

Fixes this error with chromium gpu process:
GL_INVALID_OPERATION, glBindBuffer: buffer bound to more than 1 target

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

glmemory: fix texture leak in _gl_mem_copy

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

gtk: Only run from the main thread in stop() if we created the window

We're not doing anything at all from the main thread in other cases.