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

docs: add some more Since: markers for new encoding-profile API

configure: require gobject-introspection >= 0.9.12

Earlier versions don't honour the -L/--library-path option,
which we need. See commit 4d0ccdad in gobject-introspection git.

Should "fix" build on lucid/maverick build bots.

0.10.31.3 pre-release

po: update translations

examples: allow building with newer GTK+

GtkFunction is gone, and there's no update policies for
GtkRanges any more (but the default was continuous anyway,
so no need to set it to that mode explicitly).

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

gobject-introspection: pass --library-path as well to make it find the right libgstreamer

Makes things work again properly in uninstalled setups (and
presumably in installed setups where GStreamer is installed
into a non-standard prefix). Requires fixes from core git.

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

gobject-introspection: fix issue when gold linker is used

Need to pass libgstreamer-0.10 explicitly to linker, since we're
calling gst_init(), which in turn is needed because the encoding
target get_type() function calls gst_value_register().

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

Automatic update of common submodule

From e572c87 to f94d739

Automatic update of common submodule

From ccbaa85 to e572c87

Automatic update of common submodule

From 46445ad to ccbaa85

oggdemux: remove outdated comment

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

configure: fix bash-ism

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

gobject-introspection: use same PKG_CONFIG_PATH for g-ir-compiler as for g-ir-scanner

Make sure to use the PKG_CONFIG_PATH set at configure time instead of
just relying on an env-var set one. This makes sure both g-ir-compiler
and g-ir-scanner use the same PKG_CONFIG_PATH for determining include
paths etc.

pkg-config: add girdir and typelibdir variables to .pc files

We need them when building gir and typelib files for
libraries that depend on these, such as gst-rtsp-server
for example, in an uninstalled setup.

0.10.31.2 pre-release

encodebin: Add missing-plugin support

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

encodebin: Extend documentation

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

tests: never disable g_assert() and cast checks for the unit tests

The unit tests are riddled with g_assert() and friends, sometimes
containing functional code like set_state() calls in them even
(looking at you, pipeline/capsfilter-renegotiation). Make sure we
don't disable assert and cast checks for the unit tests even if
this has been specified for the rest of the code base, e.g. via
--disable-glib-asserts.

win32: udpate pbutils .def file for API change

docs: update docs

po: update translations

textoverlay: make text property controllable too

Because we can, and because it's the most interesting one
to control really, after xpos/ypos.

textoverlay: make some properties controllable

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

tests: ignore new rtsp test binary

oggdemux: ignore header pages when looking for keyframe

This was causing keyframe_granule to be set to 0 for all streams
when seeking to the beginning of the stream, i.e., at the
beginning of playback.  Fixes #619778.

oggstream: when the last keyframe position is not known, do not use -1

Instead, use either 0 or 1, depending on bitstream version, which give
the correct result for streams which aren't cut off at start.
This allows that function to not return negative granpos.

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

Update spec file with discoverer and encodebinchanges

discoverer: Documentation updates

Some cosmetic changes and expands on some bits of the documentation to
make it more newbie-friendly.

videorate: Fix behaviour for frame rate cap changes

The outgoing buffer timestamp is calculated by scaling an output buffer
count by the src pad frame rate caps. If these caps change, we need to
reset the count and work from a new base timestamp. The new output
buffer timestamp is then the count scaled by the new caps values added
onto the base timestamp.

tools: Improve pretty-printing of tags

Avoids escaping strings for nothing and printing out useless buffer contents.