[FFmpeg-soc] [soc]: r5419 - als/alsdec.c

Thilo Borgmann thilo.borgmann at googlemail.com
Wed Oct 21 12:33:21 CEST 2009


Michael Niedermayer schrieb:
> On Tue, Oct 20, 2009 at 03:00:40PM +0200, thilo.borgmann wrote:
>> Author: thilo.borgmann
>> Date: Tue Oct 20 15:00:40 2009
>> New Revision: 5419
>>
>> Log:
>> Splits reading of block data and decoding of block data.
>> Introduces ALSBlockData struct.
> 
> You are missing the "why" part, that should be explained in the commit
> message

Yes, sorry.

> also this needs a benchmark as there are many additional dereferences
> added

It is a necessary evil to support MCC. If it would be faster the "old"
way for non-MCC files, would this reason to have both, a split read &
decode function pair and an all-in-one function?


-Thilo


More information about the FFmpeg-soc mailing list