[FFmpeg-devel] [PATCH] Fix bug in voc demuxer that was introduced in SVN r26309.

Justin Ruggles justin.ruggles
Tue Jan 18 15:50:27 CET 2011


On 01/18/2011 08:15 AM, Carl Eugen Hoyos wrote:

> Justin Ruggles <justin.ruggles <at> gmail.com> writes:
> 
>>> No, since there is a native muxer, this should be a regression test.
>>
>> That might be a good idea, especially considering our demuxer apparently
>> can't read a stereo s16 file created by our muxer.
> 
> After testing again, I'd say the muxer pretends to support s16le, but the
> resulting file is possibly invalid.


Our muxer properly supports stereo s16le, but the demuxer has a
regression in handling stereo in "old" VOC format.  The sample in this
issue is "new" VOC format so it works after the patch.  I sent a
separate patch to fix the "old" VOC stereo handling in the demuxer.

Our muxer does not write "new" VOC format for anything except
CODEC_ID_ADPCM_CT, so it still might be good to have this sample in the
fate suite.

-Justin



More information about the ffmpeg-devel mailing list