[Ffmpeg-devel] Re: [PATCH] MOV pcm sample size fix

Baptiste COUDURIER baptiste.coudurier
Sun Mar 12 20:04:49 CET 2006


Michael Niedermayer wrote:
> Hi
> 
> On Sun, Mar 12, 2006 at 04:51:39PM +0100, Baptiste COUDURIER wrote:
> 
>>Hi,
>>
>>Here is a patch to fix sample size for uncompressed sound, handling it
>>in a generic way in stsz parsing, and removing the special check in
>>read_packet.
>>
>>Is that acceptable and correct ?
> 
> 
> iam not sure, personally i would limit the code to the case where
> sample_size==1 but maybe thats not a good idea ...
> 

Well, I thought that too, but after all, if sample size is set wrongly
to 2 or something else, it will be failing, I don't really know...

Let's set it that way and see if it breaks many samples (none of mine atm) ?

-- 
Baptiste COUDURIER                              GnuPG Key Id: 0x5C1ABAAA
SMARTJOG S.A.                                    http://www.smartjog.com
Key fingerprint                 8D77134D20CC9220201FC5DB0AC9325C5C1ABAAA
Phone: +33 1 49966312





More information about the ffmpeg-devel mailing list