Tim-Philipp Müller
0d88f27108
flacparse: set both pts and dts so baseparse doesn't make up wrong dts after a seek
...
flac contains the sample offset in the frame header, so after a seek
without index flacparse will know the exact position we landed on and
timestamp buffers accordingly. It only set the pts though, which means
the baseparse-set dts which was set to the seek position prevails, and
since the seek was based on an estimate, there's likely a discrepancy
between where we wanted to land and where we did land, so from here on
that dts/pts difference will be maintained, with dts possibly multiple
seconds ahead of pts, which is just wrong. The easiest way to fix this
is to just set both pts and dts based on the sample offset, but perhaps
parsed audio should just not have dts set at all.
https://bugzilla.gnome.org/show_bug.cgi?id=752106
2015-09-06 16:36:44 +01:00
Tim-Philipp Müller
a042a98159
flacparse: use new baseparse API and fix tag handling
...
https://bugzilla.gnome.org/show_bug.cgi?id=679768
2015-08-16 16:33:55 +01:00
Thiago Santos
9523fb23ed
audioparsers: enable accept-template flag
...
Do a quick check with the pad template caps as it is enough. Users
should have figured the appropriate full caps on a previous caps query
https://bugzilla.gnome.org/show_bug.cgi?id=753623
2015-08-14 13:42:27 -03:00
Vineeth TM
e44ce40455
flacparse: fix possible memory leak
...
when buffer is stored to seektable, and stop gets called due to
corrupt flac file, then the seektable is not being released
https://bugzilla.gnome.org/show_bug.cgi?id=751364
2015-06-23 10:17:53 +02:00
Guillaume Desmottes
2bd3685d04
flacparse: fix buffer leak when stored to seektable
...
Fix a leak with the
validate.file.playback.change_state_intensive.samples_multimedia_cx_flac_Yesterday_flac
scenario.
https://bugzilla.gnome.org/show_bug.cgi?id=749072
2015-05-08 11:11:40 +01:00
Sebastian Dröge
91a3d044f0
flacparse: Only calculate with durations != -1
2014-09-23 22:56:21 +03:00
Ognyan Tonchev
7cd335e9b9
flacparse: do not leak uid after parsing TOC event
...
https://bugzilla.gnome.org/show_bug.cgi?id=736739
2014-09-17 09:51:15 +03:00
Tim-Philipp Müller
615f6e55c1
flacparse: skip PICTURE headers without any image data
...
Fixes warning if the image length is 0.
2014-05-07 00:58:15 +01:00
Vincent Penquerc'h
937269d02e
flacparse: remove dead code
...
The block_size == 0 was shortcut earlier, and the variable is not
modified in the meantime.
Coverity 206097
2014-04-16 17:44:50 +01:00
Sebastian Dröge
05e196cbb6
flacparse: Post AUDIO_CODEC tag
...
https://bugzilla.gnome.org/show_bug.cgi?id=720512
2013-12-16 10:03:06 +01:00
Wim Taymans
0d55724a2b
audioparsers: don't leak template caps
2013-12-04 09:12:07 +01:00
Wim Taymans
e0a5c07e8d
audioparsers: use ACCEPT_INTERSECT flag
...
The parser can accept input that is not completely specified. Use the
ACCEPT_INTERSECT flag on the sinkpad to tweak the acceptcaps function to
check for intersection only. This allows us to proxy downstream
constraints while still allowing non-subset caps as input.
We can then also remove the appended template caps workaround.
Make a unit-test to check the new feature.
This reverts commit 26040ee38c
Fixes https://bugzilla.gnome.org/show_bug.cgi?id=705024
2013-12-03 22:26:44 +01:00
Wim Taymans
e3f393f7e6
audioparsers: remove fields from filter
...
We need to remove the fields from the filter when we can convert
between them.
2013-12-03 21:39:57 +01:00
Wim Taymans
e8313a1e70
audioparsers: refactor code to remove caps fields
2013-12-03 21:29:13 +01:00
Sebastian Dröge
b95ddd55cd
flacparse: Make sure we have enough data to read image tags
...
Thanks to iputinei for reporting this on IRC.
2013-09-12 15:39:51 +02:00
Matej Knopp
349afc633a
flacparse: cleanup on error after state change
...
https://bugzilla.gnome.org/show_bug.cgi?id=707229
2013-09-03 18:06:18 +02:00
Matej Knopp
73751dbbe7
flacparse: Free GstBaseParseFrame if pushing a header failed
2013-09-03 10:10:49 +02:00
Sebastian Dröge
1971c43279
flacparse: Properly propagate downstream flow returns upstream
...
https://bugzilla.gnome.org/show_bug.cgi?id=707229
2013-09-02 11:56:33 +02:00
Anton Belka
0f97b6f978
flacparse: add support for the toc-select event
...
Select tracks from the CUE sheet by sending a toc-select
event based on the uid in the TOC.
https://bugzilla.gnome.org/show_bug.cgi?id=540891
2013-03-21 00:38:48 +00:00
Sebastian Dröge
1810786083
flacparse: Fix (wrong) use of uninitialized variable compiler warning
2013-03-03 11:53:04 +01:00
Sebastian Dröge
c49dede772
audioparsers: Make sure the caps are actually writable before changing them
2012-12-17 15:17:12 +01:00
Sebastian Dröge
26040ee38c
audioparsers: Use the peer caps for restrictions instead of the srcpad allowed caps
...
Otherwise we will intersect with the srcpad template caps and add all the caps fields
that the parser will ever set, no matter if downstream restricts this field or not.
This requires upstream to set this field on the caps to successfully negotiate.
https://bugzilla.gnome.org/show_bug.cgi?id=690184
2012-12-17 15:01:02 +01:00
Tim-Philipp Müller
230cf41cc9
Fix FSF address
...
https://bugzilla.gnome.org/show_bug.cgi?id=687520
2012-11-04 00:07:18 +00:00
Tim-Philipp Müller
7275860bdd
flacparse: fix coverart extraction if vorbis comments come after picture header
...
See sample file for bug #684701 .
2012-10-23 16:02:05 +01:00
Tim-Philipp Müller
7c41f42eec
flacparse: ignore bad headers if we have a valid STREAMINFO header
...
If we run into any header parsing issues and we have a valid
STREAMINFO header already, don't error out, but just stop
header parsing and try to find some audio frames.
https://bugzilla.gnome.org/show_bug.cgi?id=684701
2012-10-23 13:56:54 +01:00
Tim-Philipp Müller
49cc719809
flacparse: post proper error message and fix buffer leak on header parsing error
...
https://bugzilla.gnome.org/show_bug.cgi?id=684701
2012-10-23 13:56:54 +01:00
Mark Nauwelaerts
a2475a40a5
flacparse: fixup 0.11 port of suspect frame checking
...
Fixes https://bugzilla.gnome.org/show_bug.cgi?id=682959
2012-08-30 11:30:01 +02:00
Tim-Philipp Müller
4bb52bbadf
docs: gst-launch -> gst-launch-1.0 and ffmpegcolorspace -> videoconvert
2012-08-27 21:20:30 +01:00
Mark Nauwelaerts
2d179ebf90
flacparse: generate empty vorbiscomment for complete streamheaders if needed
...
Fixes https://bugzilla.gnome.org/show_bug.cgi?id=681335
2012-08-07 12:24:42 +02:00
Tim-Philipp Müller
c074bfd0b9
gst_tag_list_free -> gst_tag_list_unref
2012-08-04 16:10:16 +01:00
Tim-Philipp Müller
1d5ed57cfa
flacparse: update for TOC API changes
2012-07-28 11:20:08 +01:00
Sebastian Dröge
0827f54b93
tag: Update for taglist/tag event API changes
2012-07-28 00:19:51 +02:00
Sebastian Dröge
b01cf1561c
flacparse: Fix parsing of ISRC from the cuesheets
2012-07-17 10:01:54 +02:00
Anton Belka
ffc204e6bd
flacparse: add TOC support
...
Add support embedded cuesheets in flac files.
Parsing METADATA_BLOCK_CUESHEET as TOC.
https://bugzilla.gnome.org/show_bug.cgi?id=540891
2012-07-17 09:58:07 +02:00
Mark Nauwelaerts
a94d5d9f3b
flacparse: avoid some more frame misparsing by additional header sanity check
...
... using a required constant blocking_strategy bit.
https://bugzilla.gnome.org/show_bug.cgi?id=679807
2012-07-13 15:37:18 +02:00
Wim Taymans
b5df4f0e62
update for tag event change
2012-06-06 13:02:12 +02:00
Sebastian Dröge
ca4b5d795b
audioparsers: Fix GstBaseParse::get_sink_caps() implementations
...
They should take the filter caps into account and always return
the template caps appended to the actual caps. Otherwise the
parsers stop to accept unparsed streams where upstream does not
know about channels, rate, etc.
Fixes bug #677401 .
2012-06-05 09:21:08 +02:00
Edward Hervey
ba7569028c
audioparsers: Check return value of GstBitReader/GstByteReader
2012-04-12 15:47:24 +02:00
Tim-Philipp Müller
e09ae5736d
Use new gst_element_class_set_static_metadata()
2012-04-10 00:51:41 +01:00
Mark Nauwelaerts
d6cc68a9f7
audioparsers: use sink pad template caps rather than src
2012-03-22 18:27:30 +01:00
Wim Taymans
c44cd8f55b
Merge branch 'master' into 0.11
...
unport gdkpixbuf
not merged: https://bugzilla.gnome.org/show_bug.cgi?id=654850
Conflicts:
docs/plugins/Makefile.am
docs/plugins/gst-plugins-good-plugins-docs.sgml
docs/plugins/gst-plugins-good-plugins-sections.txt
docs/plugins/gst-plugins-good-plugins.hierarchy
docs/plugins/inspect/plugin-avi.xml
docs/plugins/inspect/plugin-png.xml
ext/flac/gstflacdec.c
ext/flac/gstflacdec.h
ext/libpng/gstpngdec.c
ext/libpng/gstpngenc.c
ext/speex/gstspeexdec.c
gst/audioparsers/gstflacparse.c
gst/flv/gstflvmux.c
gst/rtp/gstrtpdvdepay.c
gst/rtp/gstrtph264depay.c
2012-03-22 11:53:24 +01:00
Mark Nauwelaerts
440d7034f0
flacparse: perform additional frame crc check if applicable
...
... such as a frame header parsing throwing some suspicious warnings.
So we can be a bit more convinced we determine the right frame end.
2012-03-19 12:02:47 +01:00
Mark Nauwelaerts
58816039c2
flacparse: avoid indefinite extended search for frame end if possible
...
... which is particularly useful if locked on to the wrong frame start
and/or corrupt frame being crc checked.
2012-03-19 12:02:45 +01:00
Wim Taymans
756948262c
fix template caps refcount
2012-03-10 10:52:01 +01:00
Mark Nauwelaerts
1ae32656ae
audioparsers: adjust to modified baseparse API
2012-02-13 18:27:53 +01:00
Wim Taymans
bb2bd604e0
update for HEADER flag
2012-01-30 17:16:51 +01:00
Wim Taymans
583d39dd8d
update for new memory API
2012-01-25 12:30:28 +01:00
Tim-Philipp Müller
668e15598b
Merge remote-tracking branch 'origin/master' into 0.11
...
Conflicts:
sys/v4l2/gstv4l2object.c
2011-12-08 01:28:26 +00:00
Vincent Penquerc'h
0249a73a3e
flacparse: ensure we only check for sample/block mixup at start
...
Otherwise we might trigger at some point within the file, but the
check is only making sense for the second block.
2011-12-04 12:50:57 +00:00
Vincent Penquerc'h
c0e101e93f
various: fix pad template leaks
...
https://bugzilla.gnome.org/show_bug.cgi?id=662664
2011-11-28 13:30:27 +00:00