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

_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to