docs: convert NULL, TRUE, and FALSE to %NULL, %TRUE, and %FALSE

This should help improve documentation generated for
languages other than C.

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

docs: fix type in GstObject docs

bufferpool: fix gst_buffer_pool_has_option() documentation

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

element: set pads need-parent flag to false when removing

When a pad is added the need-parent flag is set to true, so when
they are removed the flag should be set back to false

This was preventing GstPads to be reused in elements (removed and
later re-added). A unit tests was added to verify that this is
working now.

The use case is tsdemux that has a program-number property and
allows the user to switch programs. In order to do that tsdemux
will remove the pads of the current program and add from the new
ones. The removed pads are kept in the demuxer for later if the
user selects the old program again.

multiqueue: post buffering message when queues flush

The buffering status goes back to 0, so inform the application about it

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

gitignore: Ignore VIM swap files

pad: two minor docs fixes

flowcombiner: beautify headers a little

docs: add GstFlowCombiner

base: include flowcombiner header from base.h

flowcombiner: add GstFlowCombiner

Adds a utility struct that is capable of storing and aggregating flow returns
associated with pads.

This way all demuxers will have a standard function to use and have the
same expected results.

Includes tests.

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

pad: store last flow return and provide acessor function

Stores the last result of a gst_pad_push or a pull on the GstPad and provides
a getter and a macro to access this field.

Whenever the pad is inactive it is set to FLUSHING

API: gst_pad_get_last_flow_return

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

bufferpool: Add method and virtuals to set flushing state

Currently there is no other way to unlock a buffer pool other then
stopping it. This may have the effect of freeing all the buffers,
which is too heavy for a seek. This patch add a method to enter and
leave flushing state. As a convenience, flush_start/flush_stop
virtual are added so pool implementation can also unblock their own
internal poll atomically with the rest of the pool.  This is fully
backward compatible with doing stop/start to actually flush the pool
(as being done in GstBaseSrc).

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

basetransform: Passthrough ALLOCATION queries in passthrough mode even if we had no caps yet

Or if the element does not care about caps at all.

Also remove an assigned but unused local variable.

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

po: update

po: update POTFILES

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

Back to development

Release 1.3.2

Update .po files

Automatic update of common submodule

From 211fa5f to 1f5d3c3

value: Add some positive testcase for string deserialization

docs: remove reference to Mandrake and packages we no longer provide

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

docs: fix typo

pluginloader: fix compiler warning on windows

gstpluginloader.c:584:1: error: label 'beach' defined but not used

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

elements: don't depend on libgio just for g_io_error_from_errno()

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

sparsefile: keep it private as helper API for downloadbuffer

There's no expectation that any other element or applications
might want to use this helper API any time soon, so keep it
private for the time being. There were open questions regarding
portability and binding-friendliness too.

This also removes the gio dependency of -base again.

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

docs: pick up GstBaseParse hierarchy and properties

docs: expose GstPushSrcClass in documentation

Might come in handy in case someone wants to derive from it.

pluginloader: Don't leak pluginloader in error cases

CID #1212154

caps: Don't leak features on error cases

If we fail to parse fields, we would end up leaking the features we
parsed just before

CID #1212152

basetransform: Correctly reset configuration

When pool can't we use, and we fall back to default pool, we need to
correctly reset that pool configuration.

nettimeprovider: Use non-freed variable

address is only used temporarily. Use the proper variable instead.

CID #1212189

tests: multiqueue: test to check queue overrun with pts=none

Checks if buffers with pts=none can break the queue time size limit
and allow more buffers than expected

downloadbuffer: Fix 32bit build

format '%lli' expects argument of type 'long long int', but argument 8 has type 'gsize'

pool-nego: Correctly reset the configuration

When pool cannot be used, correctly reset the configuration before
configuration a default pool.

pool-nego: Retry setting configuration with modified config

Buffer pool set_config() may return FALSE if requested configuration needed
small changes. Reget the config and try setting it again (validating the
changes first). This ensure we have a configured pool if possible.

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

bufferpool: Add an helper to validate config

When we call gst_buffer_pool_set_config() the pool may return FALSE and
slightly change the parameters. This helper is useful to do the minial required
validation before accepting the modified configuration.

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

bufferpool: Update the configure even if set_config() returned false

According to the documentation, when set_config() return false, it should be
possible to read the modified version of the config. This patch fixes the
implementation so it is now according to the documentation.

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

bufferpool: Add support for reconfiguring a pool

If a pool config is being configured again, check if the configuration have changed.
If not, skip that step. Finally, if the pool is active, try deactivating it.

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

value: Add support for GstAllocationParams comparision

This is useful to compare buffer pool configuaration.

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

value: Add support for GObject comparising in structures

This is useful to allow comparing pool configuration where a GstAllocator
is set.

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

plugin: fix case where gst_plugin_load_file() didn't set the error on failure

sparsefile: add some Since markers to docs

sparsefile: sprinkle G_BEGIN_DECLS / G_END_DECLS

for c++, and remove outdated comment, and add
new unit test to .gitignore.

downloadbuffer: small cleanups

downloadbuffer: update docs

win32: update def

Add new downloadbuffer element

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

pads: update docs for request pads

We would like to encourage the use of gst_element_request_pad()

check: use _exit() instead of exit() in fail_unless() so we exit immediately

exit() will call atexit handlers, which may try to
clean up things or wait for things to get cleaned up,
which we don't want or need. We just want to stop
and let the parent know about the failure as quickly
as possible in case fork() is used.

Fixes timeouts on assert failures in checks where
an exit handler waits for things to stop, but they
don't stop because they haven't been shut down,
and they haven't been shut down because there's no
simple way to do so on failures.

http://sourceforge.net/p/check/patches/50/

value: init flag mask more correctly

filesrc: g_memmove() is deprecated

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

Back to development

Release 1.3.1

Update .po files

po: Update translations

docs: add new device probing API to docs table of contents

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

doc: Add GstDevice* to gstreamer-sections.txt

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

Automatic update of common submodule

From bcb1518 to 211fa5f

basesink: Always render prepared buffer

Currently, if prepare() takes too much time, we skip the call to render().
The side effect of this, is that we endup starving the render(). The solution
in this patch is to always render frames that are on time before prepare() is
executed. This will maximize the number of frames we display and smoothly
degrade the rendering performance.

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

scripts/git-version.sh: add more modules

Add more git repositories to check (so git-version.sh is consistent with
gst-uninstalled) and display the date of the last commit, which is more valuable
information than the last commit's hash.

bin: Always first post the state-changed message for PAUSED->READY before posting any pending EOS message

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

basesink: Add test for checking that EOS always comes after the state change to PLAYING

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

bufferpool: fix log message of buffer pointer

scripts/git-version.sh: remove unused variable

scripts/git-version.sh: fix mistaken comments

multiqueue: avoid signaling overrun on the first segment

When the first segment has position != 0 and position > max-size-time
it will immediatelly cause the multiqueue to signal overrun.

This can happen easily with adaptive streams when switching bitrates
and starting a new group. The segment for this new group will have
a position that is much greater than 0 and will lead to this issue.
This is particularly harmful when the adaptive stream uses mpegts
that doesn't emit no-more-pads and it might happen that only one
of the stream pads was added when the multiqueue overruns and gets
the group ready for exposing. So the user will only get audio or
video.

The solution is to fallback to the sink segment while the source pad
has no segment.

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

docs: enhancements to porting guide documentation

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

query: Add boolean to URI query to specify if a redirect is permanent or not

funnel: Check if the last pad was set

If no data is coming but only EOS is sent from all of the sinkpad, it is not
forwarding the EOS.

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

docs: add docs for various GstPad macros

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

Revert "docs: add docs for various GstPad macros"

This reverts commit d17438d5fd321daec4adbeb28a8fb5d5e07298dc.

This commit featured the wrong author, sorry.

docs: remove outdated and pointless 'Last reviewed' lines from docs

They are very confusing for people, and more often than not
also just not very accurate. Seeing 'last reviewed: 2005' in
your docs is not very confidence-inspiring. Let's just remove
those comments.

buffer: Only set TAG_MEMORY if the memory has been replaced

Currently we set TAG_MEMORY as soon a resize changes the size of one
of the memory. This has the side effect that buffer pool cannot know if
the memory have simply been resized, or if the memorys has been replaced.
This make it hard to actually implement _reset(). Instead, only set the
TAG_MEMORY if one or more memory has been replaced, and do a light
sanity check of the size.

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

docs: add docs for various GstPad macros

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

bin: When going to READY make sure to always deactivate pads

We might not have reached PAUSED yet because of an async error,
but nonetheless we want to make sure that the pads are always
deactivated in READY state.

bin: Don't left-shift into the sign bit, the result is undefined

value: Use an unsigned 64 bit integer as a mask

We shift the mask to the right later and shifting the result
of shifting over the sign bit is undefined.

basesrc: Make sure to always hold the LIVE_LOCK when going to the flushing label

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

funnel: Handle end of stream event on sink pad

Handle end of stream events on sink pad. Check all the sink pad
has received eos before forwarding to source pad.

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

gstvalue: Prevent division or modulo by zero

The step can end up being zero if the underlying value isn't a valid
range GValue.

In those cases, return FALSE.

We don't use g_return*_if_fail since it will already have been triggered
by the above-mentionned _get_step() functions.

CID #1037132

libs: g-ir-scanner: do not hardcode libtool path

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

memory/buffer: Initialise GstMapInfo to zeroes if mapping fails

This should allow for more meaningful errors. Dereferencing NULL
is more useful information than dereferencing a random address
happened to be on the stack.

preset: Unref return value of gst_child_proxy_get_child_by_index() instead of leaking it

preset: Automatic code style fixes

queue2: fix event/preroll deadlock differently

The qlock is released between popping a buffer from the queue
and pushing it. When this buffer causes the sink to wait in
preroll, this lets a query see that the queue is empty, and
push the query then wait for it to be serviced. However, this
will not be done till after peroll, and this will thus block.
If upstream was waiting on buffering to reach 100% before
switching to PLAYING, a deadlock would ensue.

This had been fixed recently by failing queries when the
queue2 was buffering, but this happens to break some other
case (playbin on a local http server and matroska), while
this patch works for both.

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

check: Fix exported symbol name

it's _template and not _templ

pad: Add missing space in debug output

check: Add new API to set up pads from non-static pad templates

collectpads: Fix memory leak in unit test

testclock: replace newly-added GstTestClockIDList structure with a simple GList

Keep it simple. Likely also makes things easier for bindings,
and efficiency clearly has not been a consideration given how
the existing code handled these lists.

docs: testclock: fix up Since markers

testclock: add back gst_test_clock_wait_for_pending_id_count()

.. but deprecate it. ABI stability and all that.
It's a dangerous and racy function to use.

testclock: remove unused variable

Fixes compiler warning.

testclock: add support for waiting and releasing multiple GstClockIDs

In order to be deterministic, multiple waiting GstClockIDs needs to be
released at the same time, or else one can get into the situation that
the one being released first can add itself back again before the next
one waiting is released.

Test added for new API and old tests rewritten to comply.

pad: don't access unowned and possibly already freed event

Don't print the name of the event when ownership is given away.

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

inspect: print structure values of properties

input-selector: Hold pad object lock when changing tags

Avoid spurious crashes when tags are retrieved just as
new ones arrive.

gstbuffer: Fix range iteration

We want to iterate over items idx to idx + length

We use the len variable as the corrected number of memory to iterate
and then properly go over all items.

Fixes the issue where specifying any idx different from 0 had no effect

Spotted by clang static analyzer

structure: error out when trying to fixate a fraction near an invalid target