introspection: minor annotation additions

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

introspection: include gstversion.h in GIR generation

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

bytereader: Use concistant derefence method

This is minor style fix to not mix *var and var[N].

bytereader: Use pointer instead of index access

Currently the scan uses Boyer-moore method and its performance is good.
but, it can be optimized from an implementation of view.

The original scan code is implemented by byte array and index-based access.
In _scan_for_start_code(), the index is increasing from start to end and the
base address of the byte array is referred to as return value.

In the case, index-based access can be replaced by pointer access, which
improve the performance by removing index-related operations.

Its performace is enhanced by approximately 8% on arm-based embedded devices.
Although it seems trivial, it can affect the overall performance because the
_scan_for_start_code() function is very often called when H.264/H.265 video is
played.

In addition, the technique can apply for all architectures and it is good in
view of readability and maintainability.

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

globaldevicemonitor: prettify header

tests: add unit test for queuearray expansion from 1

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

queuearray: fix expanding size of queue from 1

Without we would not actually expand and access
memory beyond the allocated region for the array.

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

dataqueue: clear up documentation of gst_data_queue_new

The gpointer argument is passed to all three callbacks, not just one.

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

introspection: fix some minor annotation bugs

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

base: use correct syntax in documentation more consistently

Previously, many constants were prefixed with # or unprefixed,
some functions and macros were prefixed with # instead of suffixed
with (), etc.

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

baseparse: Pass rate of input segment to output segment

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

gstbuffer: factor three flags-to-string loops

info: make printing datetimes work with GST_PTR_FORMAT

datetime: change internal implementation to mini object

And move type stuff from GstValue to GstDateTime.

downloadbuffer: fix uninitialized variable

downloadbuffer: improve start/stop in buffering query

The start and stop should represent the currently downloading region.
The estimated-total should represent the remaining time to download
the currently downloading region. This makes it a lot more useful
for applications because they can then use those values to update
the fill region and use the estimated time to delay playback.
Update the docs with this clarification.

identity: add static and const where appropriate

identity: fix potential buffer overflow

Coverity 1037155

downloadbuffer: reset read and write positions

Reset the read and write positions right after we open the file or flush
it. We are also in the buffering state with 0 percent buffered when we
start.

info: first handle all miniobjects, then GObjects

First handle all miniobjects before we attempt to dereference the first
field pointer and look at the GType. With the recent glib change to
speed up G_IS_OBJECT, this causes crashes on miniobjects otherwise.

info: GstDateTime does not have a GType as first field

GstDateTime does not have the GType as the first field so we can't use
it to detect its type.

info: use macros to check types

Use the macros to check the type of objects instead of directly poking
at the first field.

globaldevicemonitor: connect sync-message signal on the right object

Fixes criticals at runtime and makes stuff actually work.

typefind: Keep still meaningfull pending events on FLUSH_STOP

Only EOS and segment should be deleted in that case.

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

Revert "miniobject: Add missing (nullable) annotations"

This reverts commit 96361e9b5c5d00dc7712ff3a9acfbe10df7cd9fe.

This was not supposed to be pushed yet!

bufferpool: It's pool, not poo... even when talking about flushing

miniobject: Add missing (nullable) annotations

gst_mini_object_replace() can take NULL mini-objects.

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

tests: multiqueue: fix leaks

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.