Sebastian Dröge
812075dc5d
flacparse: Simplify frame header parsing by using lookup tables
...
Based on a patch by Felipe Contreras.
See bug #631200 .
2010-10-07 23:38:26 +02:00
Sebastian Dröge
64407ca94b
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 .
2010-10-07 23:38:25 +02:00
Sebastian Dröge
396501784c
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.
2010-10-06 18:32:51 +02:00
Tim-Philipp Müller
716e430fd5
Revert "baseparse: add skip property"
...
This reverts commit b5a3d60363
.
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.
2010-10-05 11:17:52 +01:00
Sebastian Dröge
7e581aebe2
flacparse: Fix uninitialized variable compiler warnings
...
These warnings are wrong, the variables are only used if they were
initialized by the bit reader.
2010-10-04 10:41:52 +02:00
Felipe Contreras
b6f1e8aa68
flacparse: fix picture parsing
...
Signed-off-by: Felipe Contreras <felipe.contreras@gmail.com>
2010-10-04 08:09:57 +02:00
Sebastian Dröge
65f620af8e
flacparse: Push tags before the header buffers are pushed
2010-10-04 08:09:57 +02:00
Felipe Contreras
1a49b4de63
flacparse: trivial caps fix
...
Signed-off-by: Felipe Contreras <felipe.contreras@gmail.com>
2010-10-04 08:09:56 +02:00
Sebastian Dröge
26b2c4a1b3
audioparser: Let the format string agree with the parameters to fix compiler warning
2010-10-03 23:50:29 +02:00
Sebastian Dröge
4903e140e2
ac3parse: Use unchecked versions of the bitreader get functions
...
We didn't check the return values anyway...
2010-10-03 15:42:17 +02:00
Arun Raghavan
2d699010e9
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
2010-10-01 13:31:36 +02:00
Mark Nauwelaerts
ad1304dcf1
baseparse: support reverse playback
...
... in pull mode or upstream driven.
2010-10-01 12:15:52 +02:00
Mark Nauwelaerts
603d6ba183
baseparse: remove done TODOs and update documentation
2010-10-01 12:15:52 +02:00
Mark Nauwelaerts
fdff382686
baseparse: use determined seekability in answering SEEKING query
2010-10-01 12:15:52 +02:00
Mark Nauwelaerts
b5a3d60363
baseparse: add skip property
2010-10-01 12:15:52 +02:00
Mark Nauwelaerts
8d7720eb93
mpegaudioparse: initial version
...
... adequately equivalent to mp3parse, so lets boldly set it
to higher rank.
2010-10-01 12:15:51 +02:00
Mark Nauwelaerts
d7b31821ea
aacparse: set minimum frame size at _start
...
... rather than one time at _init.
2010-10-01 12:15:51 +02:00
Mark Nauwelaerts
36165d8d44
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.
2010-10-01 12:15:51 +02:00
Mark Nauwelaerts
13f85c8f11
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).
2010-10-01 12:15:51 +02:00
Mark Nauwelaerts
eb9ba9e115
baseparse: localize use of provided fps information
2010-10-01 12:15:50 +02:00
Mark Nauwelaerts
def0aadb81
baseparse: seek table and accurate seek support
2010-10-01 12:15:50 +02:00
Mark Nauwelaerts
174d2d46fc
baseparse: proper and more extended segment and seek handling
...
That is, loop pause handling, segment seek support, newsegment for gaps, etc
2010-10-01 12:15:50 +02:00
Mark Nauwelaerts
ec195ab2e5
baseparse: add index support
2010-10-01 12:15:50 +02:00
Mark Nauwelaerts
80646f5555
baseparse: refactor state reset
2010-10-01 12:15:49 +02:00
Mark Nauwelaerts
ad95a79de3
baseparse: prevent indefinite resyncing
2010-10-01 12:15:49 +02:00
Mark Nauwelaerts
1c14fd0a31
baseparse: specific EOS handling if no output so far
2010-10-01 12:15:49 +02:00
Mark Nauwelaerts
a1c072e0aa
baseparse: adjust _set_frame_prop documentation and set default as claimed
2010-10-01 12:15:49 +02:00
Mark Nauwelaerts
cf159041a7
baseparse: fix bitrate copy-and-paste and update heuristic
2010-10-01 12:15:49 +02:00
Mark Nauwelaerts
80ba9a1a3e
baseparse: post duration message if average bitrates is updated
2010-10-01 12:15:48 +02:00
Mark Nauwelaerts
4dff7c81d6
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.
2010-10-01 12:15:48 +02:00
Mark Nauwelaerts
c516bf8679
ac3parse: remove redundant default is_seekable
2010-10-01 12:15:48 +02:00
Mark Nauwelaerts
582b756bc1
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.
2010-10-01 12:15:48 +02:00
Mark Nauwelaerts
ea7e051827
baseparse: 0 converts to 0 by default
2010-10-01 12:15:47 +02:00
Mark Nauwelaerts
2e7de32849
basepase: refactor conversion using helper function and export default convert
2010-10-01 12:15:47 +02:00
Mark Nauwelaerts
420121705d
baseparse: streamline query handling
2010-10-01 12:15:47 +02:00
Mark Nauwelaerts
a1f51f3d17
baseparse: cleanup struct and remove unused member
2010-10-01 12:15:47 +02:00
Mark Nauwelaerts
b0d62f4182
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 ...
2010-10-01 12:15:46 +02:00
Arun Raghavan
2978216db1
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
2010-10-01 12:15:46 +02:00
Sebastian Dröge
2c6372f41a
baseparse: Don't use GST_FLOW_IS_FATAL()
...
Also don't post an error message for UNEXPECTED and do it
for NOT_LINKED.
2010-09-21 12:26:34 +02:00
Mark Nauwelaerts
a5eaa99297
baseparse: non-TIME seek event is simply not handled
2010-09-06 15:20:10 +02:00
Mark Nauwelaerts
112df6cc0e
baseparse: fix seek event ref handling
2010-06-15 15:42:26 +02:00
Mark Nauwelaerts
484af544bf
baseparse: prevent arithmetic overflows in pull mode buffer cache handling
2010-06-15 15:42:26 +02:00
Mark Nauwelaerts
07561dbfd1
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.
2010-06-15 15:42:26 +02:00
Arun Raghavan
90753ceb5e
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
2010-04-07 11:51:46 +02:00
Sebastian Dröge
09223cc719
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.
2010-03-26 18:58:35 +01:00
Sebastian Dröge
9fa738d36b
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.
2010-03-26 18:42:28 +01:00
Sebastian Dröge
6663e7c1ca
baseparse: Set the last stop to the buffer starttime if the duration is invalid
...
...instead of not setting it at all.
2010-03-26 18:20:24 +01:00
Joshua M. Doe
af00e59528
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 .
2010-03-26 18:19:00 +01:00
Arun Raghavan
cc6c4ef00b
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
2010-03-26 18:14:22 +01:00
Tim-Philipp Müller
d6f3ee4b90
audioparsers: remove unused GstBaseParseClassPrivate structure
2010-03-25 17:13:51 +00:00