[Ffmpeg-devel] [PATCH] Add fact chunk to non-PCM wav

Michel Bardiaux mbardiaux
Thu Nov 9 11:36:13 CET 2006


Michael has reported that

>> interresting, accoridng to microsofts excelent and unambigous documentation
>> (not kidding ...)
>>      Fact Chunk
>> This chunk is required for all WAVE formats other than WAVE_FORMAT_PCM. It stores file
>> dependent information about the contents of the WAVE data. It currently specifies the time length of the
>> data in samples.
>>
>> so this must not be under CODEC_ID_MSGSM, also it must be a seperate patch
>> as its not CODEC_ID_MSGSM specific

I took this literally, hence CODEC_ID_PCM_ALAW and CODEC_ID_PCM_MULAW 
will get a fact chunk too.

Note that get_wav_header does not enforce the rule on input.

Greetings,
-- 
Michel Bardiaux
R&D Director
T +32 [0] 2 790 29 41
F +32 [0] 2 790 29 02
E mailto:mbardiaux at mediaxim.be

Mediaxim NV/SA
Vorstlaan 191 Boulevard du Souverain
Brussel 1160 Bruxelles
http://www.mediaxim.com/
-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
Name: factchunk.pat
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20061109/eb9868fa/attachment.asc>



More information about the ffmpeg-devel mailing list