[SCM] libav/experimental: Set blkalign to 3840 (maximum bytes per frame) for AC-3 in avi. Fixes playback for corner-cases like 32kHz 320kb.

siretart at users.alioth.debian.org siretart at users.alioth.debian.org
Sun Jun 30 17:21:27 UTC 2013


The following commit has been merged in the experimental branch:
commit 504530bfbadfef05230179bccb0e733f032ac1a3
Author: Carl Eugen Hoyos <cehoyos at rainbow.studorg.tuwien.ac.at>
Date:   Thu Jan 6 13:30:32 2011 +0000

    Set blkalign to 3840 (maximum bytes per frame) for AC-3 in avi.
    Fixes playback for corner-cases like 32kHz 320kb.
    
    Originally committed as revision 26242 to svn://svn.ffmpeg.org/ffmpeg/trunk

diff --git a/libavformat/riff.c b/libavformat/riff.c
index 551c304..f81260a 100644
--- a/libavformat/riff.c
+++ b/libavformat/riff.c
@@ -370,13 +370,11 @@ int ff_put_wav_header(ByteIOContext *pb, AVCodecContext *enc)
         av_log(enc, AV_LOG_WARNING, "requested bits_per_coded_sample (%d) and actually stored (%d) differ\n", enc->bits_per_coded_sample, bps);
     }
 
-    if (enc->codec_id == CODEC_ID_MP2 || enc->codec_id == CODEC_ID_MP3 || enc->codec_id == CODEC_ID_AC3) {
+    if (enc->codec_id == CODEC_ID_MP2 || enc->codec_id == CODEC_ID_MP3) {
         blkalign = enc->frame_size; //this is wrong, but it seems many demuxers do not work if this is set correctly
         //blkalign = 144 * enc->bit_rate/enc->sample_rate;
-        //For high bitrate AC-3, set blkalign to maximum bytes per frame value
-        //to allow playback on WMP and MPlayer
-        if (enc->bit_rate > 384000)
-            blkalign = 3840;
+    } else if (enc->codec_id == CODEC_ID_AC3) {
+            blkalign = 3840; //maximum bytes per frame
     } else if (enc->codec_id == CODEC_ID_ADPCM_G726) { //
         blkalign = 1;
     } else if (enc->block_align != 0) { /* specified by the codec */

-- 
Libav/FFmpeg packaging



More information about the pkg-multimedia-commits mailing list