After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 653559 - aacparse: too greedy minimum frame size
aacparse: too greedy minimum frame size
Status: RESOLVED FIXED
Product: GStreamer
Classification: Platform
Component: gst-plugins-good
git master
Other All
: Normal normal
: 0.10.31
Assigned To: GStreamer Maintainers
GStreamer Maintainers
Depends on:
Blocks:
 
 
Reported: 2011-06-28 09:41 UTC by Jonas Larsson
Modified: 2011-06-28 10:44 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Change minimum frame size to ADTS_MAX_SIZE (803 bytes, patch)
2011-06-28 09:43 UTC, Jonas Larsson
committed Details | Review

Description Jonas Larsson 2011-06-28 09:41:54 UTC
Same motivation as for this commit:
http://cgit.freedesktop.org/gstreamer/gst-plugins-good/commit/?id=d17d6d74beeb8c4fd4c804d1f0b1951f3bdd06ff

"... which will be determined by parsing anyway, and avoids introducing redundant additional latency"

For low bitrate streams with initial silence the current minimum size 1024 can represent lots of frames. Mainly a problem together with video.
Comment 1 Jonas Larsson 2011-06-28 09:43:12 UTC
Created attachment 190842 [details] [review]
Change minimum frame size to ADTS_MAX_SIZE
Comment 2 Mark Nauwelaerts 2011-06-28 10:01:45 UTC
This should already have been tweaked by following commit:

commit d17d6d74beeb8c4fd4c804d1f0b1951f3bdd06ff
Author: Mark Nauwelaerts <mark.nauwelaerts@collabora.co.uk>
Date:   Mon Jun 13 17:05:19 2011 +0200

    audioparsers: not so greedy minimum frame size
    
    ... which will be determined by parsing anyway, and avoids introducing
    redundant additional latency.
Comment 3 Jonas Larsson 2011-06-28 10:06:42 UTC
I fail to see how that commit affects aacparse. It does fix ac3parse and mpegaudioparse though.

2 files changed, 7 insertions, 2 deletions
-rw-r--r--	gst/audioparsers/gstac3parse.c
-rw-r--r--	gst/audioparsers/gstmpegaudioparse.c
Comment 4 Mark Nauwelaerts 2011-06-28 10:12:44 UTC
Oops, misread that part, have been playing around with too many parsers, so it all tends to blur a bit ;)
Comment 5 Mark Nauwelaerts 2011-06-28 10:42:44 UTC
commit 1912c869469bec10dddcd26a82d18ef767228a1d
Author: Jonas Larsson <jonas.larsson@hiq.se>
Date:   Tue Jun 28 12:26:37 2011 +0200

    aacparse: not so greedy minimum frame size
    
    Fixes #653559.