[libav-devel] Split snow in snowenc snowdec

Luca Barbato lu_zero at gentoo.org
Sat Nov 19 16:08:27 CET 2011


On 19/11/11 14:51, Diego Biurrun wrote:
> On Wed, Nov 16, 2011 at 04:25:59PM +0100, Luca Barbato wrote:
>> This patchset might be squashed, it more or less show how to split a
>> large codebase in two smaller, after this set I'll try to add a reindent
>> patch over it. Please help me polish it since I'd like to make it an
>> example for the Code In refactoring tasks.
> 
> OK
> 
>> snow: split encoder from decoder
>> snow: start moving tables and common code away
>> snow: move pred_block out
>> snow: move out same_block
> 
> I wonder why you have to move out these duplicated functions.  Why didn't
> you put them in the common code to begin with?

Because I did that patchset in steps and the patch by itself is huge.

It will committed as a single one.


-- 

Luca Barbato
Gentoo/linux
http://dev.gentoo.org/~lu_zero



More information about the libav-devel mailing list