On 05/01/14 17:15, Diego Biurrun wrote: > On Sun, Jan 05, 2014 at 02:50:40PM +0100, Benjamin Larsson wrote: >> This fixes https://bugzilla.libav.org/show_bug.cgi?id=599. > > Note this in the log message. > >> From 2529a34d6f2a517ddfe27de58ed1c9352b984110 Mon Sep 17 00:00:00 2001 >> From: Benjamin Larsson <[email protected]> >> Date: Sun, 5 Jan 2014 14:42:14 +0100 >> Subject: [PATCH] flv: Workaround for buggy Omnia A/XE encoder >> >> The Omnia A/XE encoder writes the explicit extra data incorrectly. >> It disables parametric stereo wrongly, as a fix truncate the extra > > truncate_s_ > >> data by setting the size to 2. The aac extra data parser will then > > AAC > >> only parse the correct parts. > > This reads a bit strange overall, I suggest: > > The Omnia A/XE encoder writes the explicit extra data incorrectly > and wrongly disables parametric stereo. Truncating the extra data > by setting the size to 2 works around this. The AAC extra data > parser will then only parse the correct parts. > > Bug-id: 599
Fine for me. Diego, willing to push this amended version? lu _______________________________________________ libav-devel mailing list [email protected] https://lists.libav.org/mailman/listinfo/libav-devel
