baseparse: prevent arithmetic overflows in pull mode buffer cache handling

baseparse: fix seek handling

Allow a few more seek event type combinations, and really use the result
of gst_segment_set_seek to perform the seek.  Also add some debug.

baseparse: Don't emit bitrate tags too early

We wait to parse a minimum number of frames (10, arbitrarily) before
emiting bitrate tags so that our early estimates are not wildly
inaccurate for streams that start with a silence. If the stream ends
before that, we just emit the tags anyway.

While it _would_ be nicer to be specify the threshold to start pushing
the tags in terms of duration, this would introduce more complexity than
this merits.

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

baseparse: Set the last stop to the buffer starttime if the duration is invalid

...instead of not setting it at all.

baseparse: Send NEWSEGMENT event with correct start and position

Instead of taking the last stop (which could be buffer endtime instead
of starttime) always take the buffer starttime.

Fixes bug #614016.

audioparsers: remove unused GstBaseParseClassPrivate structure

audioparsers: Add bitrate calculation to baseparse

This makes baseparse keep a running average of the stream bitrate, as
well as the minimum and maximum bitrates. Subclasses can override a
vfunc to make sure that per-frame overhead from the container is not
accounted for in the bitrate calculation.

We take care not to override the bitrate, minimum-bitrate, and
maximum-bitrate tags if they have been posted upstream. We also
rate-limit the emission of bitrate so that it is only triggered by a
change of >10 kbps.

audioparsers: rename baseparse GType name to avoid possible conflicts

audioparsers: documentation fixes

baseparse: adjust seek handling and newsegment sending

Perform sanity check on type of seek, and only perform one that is
appropriately supported.  Adjust downstream newsegment event
to first buffer timestamp that is sent downstream.

baseparse: minor refactor cleanup

Also add some debug logging.

baseparse: implement leftover draining in pull mode

baseparse: provide default conversion using bps if no fps available

Also store estimated duration as such, rather than pretending otherwise
(e.g. set by subclass).

baseparse: check for remaining data when draining in push mode

baseparse: fix pull mode cache size comparison

audioparse: fix a format string as reported on irc.

baseparse: custom bufferflag indicates not to count frame in stats

audioparsers: reference GstBaseParse now lives here

audioparsers: rename 'aacparse' plugin to generic 'audioparsers' plugin

baseparse: reset passthrough mode to default (disabled) on activation

baseparse: ensure buffer metadata is writable

baseparse: fix/enhance DISCONT marking

In particular, consider DISCONT == !sync, and allow subclass to query
sync state, as it may want to perform additional checks depending
on whether sync was achieved earlier on.
Also arrange for subclass to query whether leftover data is being drained.

baseparse: add timestamp handling, and default conversion

In particular, (optionally) provide baseparse with a notion of frames per second
(and therefore also frame duration) and have it track frame and byte counts.
This way, subclass can provide baseparse with fps and have it provide default
buffer time metadata and conversions, though subclass can still install
callbacks to handle such itself.

baseparse: documentation fixes

baseparse: use_fixed_caps for src pad

After all, stream is as-is, and there is little molding to downstream's
taste that can be done.  If subclass can and wants to do so, it can
still override as such.

aacparse: Fix compilation warnings

aacparse: fix warnings in macosx snow leopard

aacparse: forego (bogus) parsing of already parsed (raw) input

baseparse: prevent infinite loop when draining

baseparse: fix minor memory leak

aacparse: Add function for the baseparse subclass to push buffers downstream

Also handle the case gracefully where the subclass decides to drop
the first buffers and has no caps set yet. It's still required to
have valid caps set when the first buffer should be passed downstream.

baseparse: Fix seek event leaking

baseparse: propagate return value of GstBaseParse::set_sink_caps()

gst_base_parse_sink_setcaps() presumably should fail if the subclass
returns FALSE from its ::set_sink_caps() function.

baseparse: don't try to GST_LOG an already-freed caps string

The proper way to log caps is via GST_PTR_FORMAT anyway.

baseparse: fix debug category

baseparse: fix (regression in) newsegment handling

(aacparse, amrparse, flacparse).  Fixes #580133.

baseparse: Fix slightly broken buffer-in-segment check (aacparse, amrparse, flacparse)

baseparse: Fix push mode seeking (aacparse, amrparse)

Sending the flush-start event forward before taking the stream lock actually
works, in contrast to deadlocking in downstream preroll_wait (hunk 1).

After that we get the chain function being stuck in a busy loop. This is fixed
by updating the minimum frame size inside the synchronization loop because the
subclass asks for more data in this way (hunk 2).

Finally, this leads to a very probable crash because the subclass can find a
valid frame with a size greater than the currently available data in the
adapter. This makes the subsequent gst_adapter_take_buffer call return NULL,
which is not expected (hunk 3).

baseparse: Delay newsegment as long as possible.

If newsegment is sent (too) early, caps may not yet be fixed/set,
and downstream may not have been linked.

baseparse: revert last change and properly fix

Baseparse internaly breaks the semantics of a _chain function by calling it with
buffer==NULL. The reson I belived it was okay to remove it was that there is
also an unchecked access to buffer later in _chain. Actually that code is wrong,
as it most probably wants to set discont on the outgoing buffer.

baseparse: remove checks for buffer==NULL

Accordifn to docs for GstPadChainFunction buffer cannot be NULL. If we would
leave the check, we would also need more such check below.

Fix the return value of the default parse_frame function.

Fix the return value of the default parse_frame function in both
copies of GstBaseParse

gst/: Fix baseparse type name.

Original commit message from CVS:
* gst/aacparse/gstbaseparse.c:
* gst/amrparse/gstbaseparse.c:
Fix baseparse type name.

Add two new baseparse based parsers (aac and amr) from Bug #518857.

Original commit message from CVS:
* configure.ac:
* gst/aacparse/Makefile.am:
* gst/aacparse/gstaacparse.c:
* gst/aacparse/gstaacparse.h:
* gst/aacparse/gstbaseparse.c:
* gst/aacparse/gstbaseparse.h:
* gst/amrparse/Makefile.am:
* gst/amrparse/gstamrparse.c:
* gst/amrparse/gstamrparse.h:
* gst/amrparse/gstbaseparse.c:
* gst/amrparse/gstbaseparse.h:
Add two new baseparse based parsers (aac and amr) from Bug #518857.

queue[2]: Make src query MT-safe

It is possible that the element might be going down while the event arrives

basesrc: Make src query MT-safe

It is possible that the element might be going down while the event arrives

queue[2]: Unref events if the parent element disappeared

queue[2]: Make upstream events MT-safe

base: Unref events if the parent element disappeared

And also unref events if the basetransform subclass has no
event handler and the event is not forwarded at all.

base: Make upstream events MT-safe

gstqueue, gstqueue2: check if parent of pad is NULL in _getcaps

Parent of the pad (the queue) may be set to NULL while there is
a buffer alloc going on.

inputselector: Fix getcaps and event function from last commit

Return ANY caps if the parent disappeared, i.e. the template caps
and don't leak events if the parent disappeared.

inputselector: Protect against pad-parent disappearing

iterator: resync to avoid infinite loop

utils: Fix uninitialized variable compiler warnings

bus: Removed unused GCond

multiqueue: Add another check for the flushing flag after taking the lock

This prevents another potential deadlock when flushing the pad
at exactly the right time.

bus: Immediately drop messages after calling the sync handler if this is a synchronous bus

Otherwise we might wait forever for the message to be popped from
the queue if a sync handler returned GST_BUS_ASYNC.

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

bus: Use a construct-only property to distinguish between child buses and normal buses

This allows to only create the socketpair when it is really required instead
of always creating it and immediately destroying it again for child buses.

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

tests: add some basic unit tests for queue2

queue2: fix buffer leak on eos when using the ring buffer

queue2: Fixes memory leak on out_flushing error in gst_queue2_create_read.

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

queue2: fix minor memory leak

funnel: minor element description fix

docs: remove file as we have docs/design/part-missing-plugins.txt

basesrc: Handle tag and custom downstream events the same

Especially drop tag events when flushing to not send them over
and over again.

Should've been in the last commit already but I forgot to call
git rebase --continue...

bla

basesrc: Send syncronized custom downstream/both events downstream from the streaming thread

Instead of just silently dropping them. The same was done for tag events
before already.

Fixes bug #635718.

bus: Add private API to set a GstBus in child mode

This is used by GstBin to create a child bus without
a socketpair because child buses will always work
synchronous. Otherwise too many sockets could be
created and the limit of file descriptors for the
process could be reached.

Fixes bug #646624.

Revert "bus: Only create the signalling socket pair when required"

This reverts commit 4bf8f1524f6e3374b3f3bc57322337723d06b928.

Revert "bus: Check if pending messages are in the queue"

This reverts commit bd1c40011434c1efaa696dc98ef855ef9cce9b28.

checks: make tests_many_bins in bin unit test a bit faster

Not doing expensive checks when linking elements makes things
much faster.

checks: add some queues to test_many_bins unit test

To limit the number of calls in a row per thread.

bus: Check if pending messages are in the queue

We can't rely completely on the poll fd because the fd might be
created after messages were posted to the bus.

value: GstDate/GDate has a abbreviation now

checks: add GstBin unit test that creates a lot of bins

Currently fails (in normal circumstances) because we create a
socket pair for each bin's bus and exhaust the number of available
file descriptors.

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

bus: Only create the signalling socket pair when required

Otherwise a new one would be created for every single bus and
the process could easily run out of file descriptors.

Fixes bug #646624.

structure: Add date as a type abbreviation of GDate

See bug #646696.

Automatic update of common submodule

From 1ccbe09 to c3cafe1

gstpoll: retry reading the control socket to release properly all wakeups

if set->control_pending is set to 0 but we didn't not succed reading
the control socket, future calls to gst_poll_wait() will be awaiken
by the control socket which will not be released properly because
set->control_pending is already 0, causing an infinite loop.

structure: Don't allow invalid GDates in all structures and don't allow NULL GDates in taglists

Some code (e.g. gstvorbistag.c) assumes non-NULL GDates in taglists and
explodes otherwise and NULL or invalid GDates don't make much sense anyway.

poll: don't call WSAWaitForMultipleEvents with no events

Fixes error caught by Microsoft Application Verifier.

docs: add pointer to part-seeking.txt design docs to event seek flags docs

checks: ignore new funnel unit test binary

utils: Avoid using "type" as name for a variable and a macro argument in GST_BOILERPLATE

This caused "re-declaration" problems.
./clutter-gst-video-sink.c: In function ‘clutter_gst_video_sink_init_interfaces’:
./clutter-gst-video-sink.c:231:1: warning: declaration of ‘ClutterGstVideoSink’ shadows a global declaration [-Wshadow]
./clutter-gst-video-sink.h:64:44: warning: shadowed declaration is here [-Wshadow]

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

element: When requesting an existing pad print a g_critical() instead of using an assertion

Some applications are requesting the same pad name multiple times
and the behaviour is undefined and different from element to element
but we don't want to break applications that work just fine.

In 0.11 this check should be an assertion again, although elements
have to do manual checking if the pad already exists again because
it can't be done in a threadsafe way here.

element: Use gint64/guint64 string parsing functions

And check that the requested pad names are inside the valid
gint/guint range.

element: strto[u]l() returns a g[u]long

design docs: document expected behaviour for ACCURATE and KEY_UNIT seek flags

gstabi: Add some new structures for x86-64

libsabi: Add lots of new structures for x86-64

multiqueue: Make assignment of queue IDs and pad names threadsafe

Also add a test for naming pads by the caller and return NULL
when requesting an already existing pad.

multiqueue: Set the single queue ID to the ID of the requested pad if one was given by the caller

element: Fix sanity checks for request pad templates without %

element: Add some more sanity checks to the pad name checking of request pads in all cases

Especially check if a pad with that name already exists.

element: Check %u too when trying to find a pad template for a request pad

multiqueue: Fix arbitrary sink + source pad naming

Use the string provided by the caller for the sinkpad name
if possible. Note that all sanity checking for this name
is already done in GstElement.

Fixes Bug #645931

funnel: Add some more documentation about the behaviour of funnel

funnel: Send a newsegment event after flush-stop

funnel: Some random cleanup