audioparsers: add back segment clipping to parsers that have lost it
authorMart Raudsepp <mart.raudsepp@collabora.com>
Mon, 24 Jun 2019 11:28:39 +0000 (14:28 +0300)
committerMart Raudsepp <mart.raudsepp@collabora.com>
Mon, 24 Jun 2019 11:40:58 +0000 (14:40 +0300)
commitab6e49e9cc398dcd27a39776fb7bdfe4889cb8e8
treead54454fe69f02f18b13866343dc4360cfeb509f
parent92e4ecef4caa09f0caa83a8440d0d641b75f9e2f
audioparsers: add back segment clipping to parsers that have lost it

The pre_push_frame default clipping behaviour was introduced in 2010
with commit 30be03004e82 and modified with commit 4163969a2422 in 2011,
when most parsers didn't implement a pre_push_frame yet. Not having it
meant that clipping was done by default. Those that did implement a
pre_push_frame (flacparse and mpegaudioparse) at the time, had the flag
adjusted as part of the 2011 refactor work.

All other parsers got a pre_push_frame vfunc implementation only in
2013, but seem to have forgot to keep the clipping behaviour, as
was done automatically when a pre_push_frame implementation doesn't
exist for the parser. aacparse lost it with commit 91d4abcea in
July 2013; the others in Dec 2013 as part of AUDIO_CODEC tag posting
in commits 6f89b430ed2ab5199b29f2cae12753d3c23a and 292780574.
gst/audioparsers/gstaacparse.c
gst/audioparsers/gstac3parse.c
gst/audioparsers/gstamrparse.c
gst/audioparsers/gstdcaparse.c
gst/audioparsers/gstsbcparse.c
gst/audioparsers/gstwavpackparse.c