encodebin: Return a new reference of the pad for the "request-pad" signal

The GObject signal code assumes that the signal handlers return a
new reference or copy. Fixes bug #641927.

adder: Fill in offset_end field of outgoing buffers

... rather than leave it as GST_BUFFER_OFFSET_NONE

Fix bug #642942.

playsink: release all chains when going to NULL

Also fixes #642466.

playsink: undo state change side effect on error way out

... to avoid subsequent cleanup disposing an element not in NULL state.

playsink: avoid crashing on the way out when needed chain missing

win32: update .def file for new libgstvideo API

discoverer: handle desc==NULL

It would otherwise be printed as (null) and mess up indentation (no \n).

discoverer: Chain dispose() up to parent class

discoverer: Keep a ref for the async timeout callback

This makes sure we maintain a ref on the discoverer object while the
async timeout callback is alive to prevent a potential crash if the
object is freed while the callback is pending.

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

discoverer: Use g_signal_connect_object instead of g_signal_connect

We want to make sure the discoverer object passed to the various
callbacks doesn't become invalid if a callback is pending and the object
is free'd in the mean time.

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

typefinding: detect raw h.263

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

pbutils: encoding-target: fix error checking in target file loading

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

tests: fix videoscale test by ignoring newly-added 64-bit formats

They probably fail because ffmpegcolorspace can't handle those formats.

sdp: Fix copy/paste error in inrospection part of Makefile

tag: Fix copy/paste error in inrospection part of Makefile

rtsp: Fix copy/paste error in inrospection part of Makefile

audiorate: add skip-to-first property

API: GstAudioRate::skip-to-first

videorate: fix skip-to-first ts setup

... such as avoiding arithmetic mixing counts and ts, although latter
would typically be 0 so far.

Revert "oggmux,adder: Check if collectpads has been freed"

This reverts commit 6d150873e8b4c23d694b0351570de323b1576d76.

Depends on a core commit that was reverted.

oggmux,adder: Check if collectpads has been freed

Core now calls release_pad in finalize, which is usually after
the collectpads has been unreffed.

videoscale: Add 16-bit-channel support

videotestsrc: Add 16-bit-per-channel formats

video: Add ARGB64 and AYUV64

16-bit per channel formats.

video: Add gst_video_format_get_component_depth()

audiotestsrc: each element gets its own instance of GRand, if needed

As a result, pipelines that contain multiple instances of audiotestsrc
with the 'wave' property set to 'white-noise', 'pink-noise', or
'gaussian-noise' will run much faster, since they won't be competing
for access to the global, lock-protected instance of GRand.

Fixes bug #642720.

playbin2: If a sink claims to support ANY caps assume that it only supports the usual raw formats

This should be changed again in 0.11, if a sink really claims to support ANY
caps it should support everything or provide correct caps.

encodebin: Add a audioconverter after the audio resampler.

This allows handling non-native-endianness conversion properly.

playbin2: Use gst_pad_accept_caps() instead of intersecting with the getcaps caps

This might be faster and more accurate in some cases to detect if a
sink supports a format and autoplugging can be stopped.

uridecodebin: Add default handler for autoplug-select

uridecodebin proxies this signal and only the first signal handler
will ever be called from decodebin2, which is uridecodebin's proxy
signal handler.

uridecodebin: Return NULL from the default autoplug-sort handler

...instead of copying the array. Returning NULL will result
in the original factories array to be used and prevents a useless
array copy in most use cases.

decodebin2: Return NULL from the default autoplug-sort handler

...instead of copying the array. Returning NULL will result
in the original factories array to be used and prevents a useless
array copy in most use cases.

uridecodebin: Update autoplug-* signal docs from decodebin2

uridecodebin proxies these signals.

decodebin2: Update documentation of the autoplug-* signals

Add notes about the behaviour if multiple signal handlers are connected.
For most autoplug-* signals only the first signal handler will ever
be invoked.

Also add to the autoplug-sort docs that the signal handler can return NULL
to specify that the order should change and other handlers get the chance
to sort the array.

decodebin2: Keep the original factory list if the sort signal handlers returned NULL

uridecodebin: expose "autoplug-sort" signal

It is a proxy of the decodebin2's one, and was missing
in the previous code.

See bug #642433.

playbin2: Use a recursive mutex for the playbin lock

This lock is taken when activating a group, which could result in
calling the autoplug-continue callback, which also needs this lock
to access the sinks.

See bug #642174.

decodebin2: Disconnect signal handlers when removing a failed element

This prevents crashes later if one of the signals is emitted after the
element was removed from decodebin2 already, which can happen in discoverer.

typefind: Fix mpeg TS detection

theoraenc: move debug category init earlier

oggparse: better detection of delta unit flag

theoraenc: Set speed level while running

Set the theoraenc speed-level property from libtheora's defaults.

The speed-level property, which allows callers to trade of encoding
quality for speed in the libtheora api, has a version-dependent
maximum and default values. Instead of hardcoding the acceptable
range for the theoraenc element's presentation of this setting,
we query the library directly at class initialization time and
set the maximum and default values from that. If the query fails,
we fall back to the previous default setting.

To keep the values reported by gst-inspect (which I'm told use
the spec values from the class) with those available on an\
instantiated element, we remove to setting of enc->speed_level
from the initializer and instead pass G_PARAM_CONSTRUCT to
the property spec flags, asking g_object to set this property
when theoraenc objects are constructed.

NB in theory the maximum speed-level could depend on the actual
video caps. If later versions of libtheoraenc do this, a second
call will need to be made from theora_enc_reset to update the
property, since this function is mostly useful for realtime
adjustment of performance while the pipeline is running.

discoverer: don't leak parent tags

discoverer: improve logging (and reindent)

Add more logging for the tag merging and use the _OBJECT flavour more.

playbin2: Optimize autoplug-continue handler a bit

Don't build merge the caps of all sinks but check them one-by-one
until one supports the caps. Also get reffed caps from the sinkpads
instead of a writable copy and add debug output if a sink claims to
support ANY caps.

playbin2: Fix handling of non-raw custom sinks

When autoplugging elements in decodebin2, check if
the caps are supported by one of the sink before
continuing autoplugging.

Fixes bug #642174.

decodebin2: Don't leak elements that fail to go to PAUSED after being autoplugged

Fixes bug #642381.

theoraenc: Don't reset the video quality setting the bitrate

libtheora has two encoding modes, CBR, where it tries to hit a target
bitrate and VBR where it tries to achieve a target quality.

Internally if the target bitrate is set to anything other then 0 the
encoding-mode is CBR.

This means that the gstreamer element can leave the video_quality
setting alone as long as the user is tweaking the bitrate. Which has the
nice side-effect that if the user explicitely sets the bitrate to 0
(which is actually the default), the quality value doesn't get reset and
one ends up encoding VBR at quality-level 0...

gdppay: ensure buffer's metadata is writable before setting caps

Automatic update of common submodule

From f94d739 to 1de7f6a

doap: update mailing list location

discoverer: Use nominal bitrate if bitrate tag is unavailable

If the bitrate tag is unavailable, this falls back to the nominal
bitrate tag instead, if that is present.

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

decodebin2: caps can be NULL

Don't use and unref NULL caps.

tagdemux: also push cached events downstream when operating in pull mode

Otherwise, having 2 tagdemux in a row followed by an element operating in
pull mode will make the second tagdemux implictly eat the first tagdemux'
tag event(s).

Fixes (part of) #641047.

oggmux: ensure serialnos are unique

We do that by checking a newly generated one is not already used in
an existing stream, and doing it again if it is.

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

oggmux: free stream map caps when done

oggmux: keep IN_CAPS flag check for header buffers as fallback

In case the ogg mapper doesn't handle all the accepted input formats
(although it really should). Saves us error handling for that case
though. Also log caps properly.

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

oggmux: use oggstream for less brittleness in recognizing headers

Using the IN_CAPS flag for this is brittle, and will fail if either
vorbisparse or vorbistag (which is itself based on vorbisparse) is
inserted between oggdemux and oggmux. Possibly other elements too
(eg, theoraparse, etc).
Using oggstream ensures we Get It Right More Often Than Not.

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

uridecodebin: fix copy-and-paste typo in property docs

oggmux: do not skip a pageno at start

Discontinuities are automatically signalled by oggdemux at the start
of a new stream. When oggmux is yet to output actual data pages,
do not signal these discontinuities in the ogg stream.

This patch may miss some actual discontinuities at the very start of
a stream, but avoids the spurious missing pages when encoding happens
normally.

A better fix might involve finding a way to distinguish between actual
data discontinuities and discontinuities merely marking the start of
a new stream.

Fixes an issue with ogg page numbering (would skip a number for no
reason, which then looks like a packet was lost somewhere) when
re-muxing an ogg stream, e.g. when re-tagging in rhythmbox.

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

theoraenc: clean up property descriptions

Remove "This property requires libtheora version >= 1.1" qualifiers
from property descriptions. They aren't needed any longer now that
we require libtheora >= 1.1.

id3tag: map the ID3v2 TENC frame to GST_TAG_ENCODED_BY

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

tcp: use socklen_t where appropriate rather than specific type

In particular, fixes Cygwin build where socklen_t is defined as int
in line with native win32 api definition.

xmptag: cast argument to isdigit to int

... as that is the specification and fixes compilation on Cygwin:

gstxmptaag.c: In function 'read_one_tag':
gstxmptag.c:1015: error: array subscript has type 'char'

appsink: add buffer fallback in case the application doesn't handle buffer lists

We shouldn't assume the application handles buffer lists, for
ease-of-use reasons and for backwards compatibility reasons.

appsink: send new-buffer-list signal

Send new-buffer-list signal when emit-signals is TRUE

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

uridecodebin: also add https to buffer protocols

HTTPS also needs buffering.

baseaudiosink: trivial cleanups

It seems these stuff was neglected from commmit d8942e2.

Signed-off-by: Felipe Contreras <felipe.contreras@nokia.com>

win32: fix DEFAULT_AUDIOSINK, should be direct*sound*sink

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

typefinding: register H264 typefinder with H264 caps

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

streamsplitter: release pending events refs

Unref pending events when disposing the streamsplitter.

Also refactor a little to replace a for with a g_list_foreach

tests: don't run encodebin test if vorbis or theora plugins aren't available

pbutils: add description for degas images

typefinding: use image/x-icon as media type for ICON files

That's what we've been using so far (e.g. gdkpixbufdec).

snapshot: use a keyframe seek

One would usualy get good quality snapshots quickly. The exact seek position
does not really matter.

snapshot: add a newline to the usage and error output

decodebin2: add comment and whitespace trimming

pbutils: add description for x-annodex

typefind: add typefinder for DEGAS images

This fixes at least one DEGAS image from being misdetected as DTS audio.

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

oggmux: cleanup

Remove a pointless string concatentation, and fix an off-by-one in
packetno in a log.

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

typefind: add detection for windows icon files to get them out of the way

Some of them can otherwise be misdetected for MPEG audio.

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

oggdemux: Remove dead code

oggparse: Make gst_ogg_parse_submit_buffer() safe

By not passing zero-sized buffers to ogg_sync_buffer()
and checking the return values of libogg functions.

Fixes bug #639136.

textoverlay: Add support for vertical center alignment

Fixes bug #639159.

tag: xmp: Move static variable to local function

Variable was being written to and could cause crashes
if multiple elements were parsing xmp at the same time.

Moving it to local scope solves the problem.

riff: Add support for video/x-camstudio

Back to development

Release 0.10.32

0.10.31.4 pre-releases

docs: update docs

encoding-target: change keyfile header to 'GStreamer Encoding Target'

which is more in line with other files such as .desktop files.

pbutils: don't assume LC_MESSAGES is always defined, also check for ENABLE_NLS

Should fix build with mingw32 build bot again.

app: export gst_app_stream_type_get_type()

API: gst_app_stream_type_get_type()
API: GST_TYPE_APP_STREAM_TYPE

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

app: make GstAppBuffer get_type() function thread-safe

discoverer: Drop new stream tags once preroll is done

This makes sure we do not touch the stream taglist once the pipeline has
been prerolled. Adding of stream tags happens in the pad event probe
which runs in a different thread from discoverer stream processing, so
modifying the tag list while discoverer might be processing it can
sometimes cause a crash.

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

discoverer: Validate timeouts before processing them

This avoids a race where the timeout callback is scheduled to run but we
get sufficient information to finish discovery before actually getting
around to executing the callback. See the documentation of
g_source_is_destroyed() for more details.

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

discoverer: Make sure we call _stop() before being freed

This ensures that everything is properly cleaned up before the
GstDiscoverer object is freed. Specifically, it makes sure that we've
removed the async timeout callback before freeing the object to avoid a
potential crash later on.

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

gdppay: make newsegment buffer metadata writable

pbutils: save localised strings properly when writing encoding targets to a file

Use LC_MESSAGES rather than LC_ALL. Save/load description as untranslated string
when using an English language locale. Strip locale information to the language,
so we don't save keys like description[fr_FR.UTF-8]=...

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

typefinding: set framed=false on DTS caps