baesparse: fix refactor regression in loop based parsing

baseparse: pass all available data to subclass rather than minimum

Also reduce some adapter calls and add a few debug statements.

baseparse: fix reverse playback handling

baseparse: minor typo and debug statement cleanup

baseparse: reduce locking

... which is either already mute and/or implicitly handled by STREAM_LOCK.

baseparse: avoid loop in frame locating interpolation

flacparse: mind gst_buffer_unref not liking NULL

Fixes #639950.

audioparsers: baseparse: Be careful to not lose the event ref

Don't unref the event if it hasn't been handled, because the caller
assumes it is still valid and might reuse it.

I ran into this problem when transcoding an AVI (with mp3 inside)
to gpp.

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

dcaparse: fix sync word for 14-bit little endian coding

Fix copy'n'paste bug that made us look for the raw little endian
sync word twice instead of looking for the 14-bit LE sync word
as well. Fixes parsing of such streams (see #636234 for sample file).

docs: minor baseparse docs/comment fixes

Remove copy'n'paste leftovers.

flacparse: Fix unitialized variable on macosx

ac3parse: relax bsid checking

... to the widest possible spec interpretation.

Fixes #637062.

audioparsers: update some documentation

mpegaudioparse: add to documentation

dcaparse: add to documentation

baseparse: increase keyframe awareness

... which is not particular relevant for audio parsing, but more so
in video cases.  In particular, auto-determine if dealing with video (caps).

ac3parse: use proper EAC-3 caps

baseparse: avoid unexpected stray metadata

baseparse: use proper _NONE output value when applicable

audioparsers: Remove dead assignments

audioparse: fix possible division-by-zero

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

baseparse: use correct offset when adding index entry

... bearing in mind that BUFFER_OFFSET is media specific and may not
reflect the basic offset after having been parsed.

baseparse: enhancements for timestamp marked framed formats

That is, as such formats allow subclass to extract position from frame,
it is possible to extract duration (if not otherwise provided)
from (near) last frame, and a seek can fairly accurately target the required
position.

Fixes #631389.

baseparse: refactor frame scanning peformed by _loop

baseparse: slightly optimize sending of pending newsegment events

baseparse: minor fixes and enhancements

Arrange for upstream as well as downstream flushing when seeking.
Also determine upstream size as well as seekability.  Adjust some comments
to reality and employ debug statement in proper order.

aacparse: minor cleanups

aacparse: fix regression in ADIF src caps setting

flacparse: parse seektable

Fixes #631389 (partially).

flacparse: minor refactor and enable default baseparse segment clipping

mpegaudioparse: fix silly leak in _reset

baseparse: use only upstream duration if it provides one

baseparse: reflow update_bitrate code

... which makes local variables represent real state better, and avoids
triggering unneeded updates/actions.

baseparse: add some debug statements

dcaparse: init variable to make osx build bot happy

gstdcaparse.c: In function 'gst_dca_parse_check_valid_frame':
gstdcaparse.c:246: warning: 'best_sync' may be used uninitialized in this function

audioparsers: add very basic dts/dca parser

Still some issues, e.g. with seekable queries in totem, but also
processing already-chunked input (created with matroskademux ! gdppay).

ac3parse: properly parse e-ac3 frame header

Also add a few debug statements.

flacparse: tweak setting buffer metadata; avoid timestamp jitter

Fixes #631993.

aacparse: streamline src caps setting

In particular, also set src caps whenever changes in stream warrant doing so.

flacparse: Adjust unit tests to new flacparse behaviour

Garbage after frames is now included in the frames because flacparse
has no easy way to detect the real end of a frame. Decoders are
expected to everything after the frame because only decoding the
bitstream will reveal the real end of the frame.

Fixes bug #631814.

flacparse: Don't drop the last frame if it is followed by garbage

See bug #631814.

baseparse: perform bitrate handling and posting after newsegment sending

baseparse: immediately post subclass provided bitrate

flacparse: fix parsing with unknown framesizes

Fixes #631814 (mostly).

flacparse: Simplify frame header parsing by using lookup tables

Based on a patch by Felipe Contreras.

See bug #631200.

flacparse: Don't parse the complete FLAC frames but only look for valid frame headers

Thanks to Felipe Contreras for the suggestion. This is partially
based on his patches and makes flacparse more than 3.5 times faster.

Looking for valid frame headers is unlikely to give false positives
because every frame header is at least 9 bytes long, contains a
14 bit sync code and a 8 bit checksum over the first 8 bytes.

Fixes bug #631200.

flacparse: Really post tags only after the initial newsegment event

The first newsegment event will be send by the first call to
gst_base_parse_push_buffer() if necessary, posting the tags
before that is not a good idea. Instead do it from the
GstBaseParse::pre_push_buffer vfunc.

Revert "baseparse: add skip property"

This reverts commit b5a3d60363d837a10f0533c141ec93d10b742312.

Reverting this for now, since no one really seems to remember why this
property exists or what it could possibly be good for. It seems to have
been in the original mp3parse since the beginning of time and was back-
ported from there.

flacparse: Fix uninitialized variable compiler warnings

These warnings are wrong, the variables are only used if they were
initialized by the bit reader.

flacparse: fix picture parsing

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

flacparse: Push tags before the header buffers are pushed

flacparse: trivial caps fix

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

audioparser: Let the format string agree with the parameters to fix compiler warning

ac3parse: Use unchecked versions of the bitreader get functions

We didn't check the return values anyway...

baseparse: Fix debug output

We lose the reference to the buffer after gst_pad_push(), so the debug
print should happen before.

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

audioparsers: add flacparse unit test

... and tweak parser test helper in the process.

baseparse: support reverse playback

... in pull mode or upstream driven.

baseparse: remove done TODOs and update documentation

baseparse: use determined seekability in answering SEEKING query

baseparse: add skip property

audioparsers: add ac3parse and mpegaudioparse unit test

mpegaudioparse: initial version

... adequately equivalent to mp3parse, so lets boldly set it
to higher rank.

aacparse: set minimum frame size at _start

... rather than one time at _init.

audioparsers: refactor existing unit tests using common helper

baseparse: use _set_frame_props to configure frame lead_in and lead_out

... provided a corresponding decoder with sufficient leading and following
frames to carry out full decoding for a particular segment.

baseparse: use _set_duration to configure duration update interval

... as it logically belongs there as one or the other; either subclass
can provide a duration, or an estimate must be made (reguarly updated).

baseparse: localize use of provided fps information

baseparse: seek table and accurate seek support

baseparse: proper and more extended segment and seek handling

That is, loop pause handling, segment seek support, newsegment for gaps, etc

baseparse: add index support

baseparse: refactor state reset

baseparse: prevent indefinite resyncing

baseparse: specific EOS handling if no output so far

baseparse: adjust _set_frame_prop documentation and set default as claimed

baseparse: fix bitrate copy-and-paste and update heuristic

baseparse: post duration message if average bitrates is updated

baseparse: remove is_seekable vmethod and use a set_seek instead

Seekability, like duration, etc is unlikely to change (frequently), and
the default assumption covers most cases, so let subclass set when needed.
At the same time, allow subclass to indicate if it has seek-metadata (table)
available, and possibly have it provide an average bitrate.

ac3parse: remove redundant default is_seekable

baseparse: add another hook for subclass prior to pushing buffer

... and allow subclass to perform custom segment clipping, or to
emit tags or messages at this time.

baseparse: 0 converts to 0 by default

basepase: refactor conversion using helper function and export default convert

baseparse: streamline query handling

baseparse: cleanup struct and remove unused member

audioparsers: increase ranks to enable auto-plugging

Because we can, and should, have some shakedown testing before having
these make it into -good later on ...

baseparse: Allow chaining of subclass event handlers

This allows the child class to chain its event handler with
GstBaseParse, so that subclasses don't have to duplicate all the default
event handling logic.

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

baseparse: Don't use GST_FLOW_IS_FATAL()

Also don't post an error message for UNEXPECTED and do it
for NOT_LINKED.

baseparse: non-TIME seek event is simply not handled

baseparse: fix seek event ref handling

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.

check: Don't re-declare 'GList *buffers' in the tests

It's an external which lives in gstcheck.c. Redeclaring it makes some
compilers/architectures think the 'buffers' in the individual tests are
a different symbol... and therefore we end up comparing holodecks with
oranges.

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

flacparse: Optionally check the overall frame checksums too before accepting a frame as valid

This is optional because it's a quite expensive operation and it's very
unlikely that a non-frame is detected as frame after the header CRC check
and checking all bits for valid values. The overall frame checksums are
mainly useful to detect inconsistencies in the encoded payload.

flacparse: Check the CRC-8 of the headers before accepting a frame as valid

This makes false-positives during seeking much less likely and detection of
them much faster.

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.

flacparse: Fix buffer refcount issue

When called from the GST_FLAC_PARSE_STATE_HEADERS case,
gst_flac_parse_hand_headers() does a gst_buffer_set_caps() on a buffer
with refcount > 1. This change handles this case by making the buffer
metadata_Writable.

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

audioparsers: remove unused GstBaseParseClassPrivate structure

flacparse: Make bitrate estimation more accurate

This implements the get_frame_overhead() vfunc so that baseparse can
make more accurate bitrate estimates.

aacparse: Fix bitrate calculation

This patch adds the get_frame_overhead() vfunc so that baseparse can
accurately calculate the min/avg/max bitrates for aacparse.

Note: The bitrate was being incorrectly calculated for ADTS streams
(it's not in the header as the code suggests).