On 11/16/2011 10:25 AM, 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.
> 
> snow: split encoder from decoder
> snow: start moving tables and common code away
> snow: move pred_block out
> snow: move out same_block
> snow: move add_yblock out
> snow: move away predict_plane
> snow: move away common_init_after_header
> snow: remove tree_sse and QUANTIZE2 related code
> snow: move remaining common code away
> snow: move away the remaining shared inline functions


When you're moving functions from the encoder/decoder to the shared
snow.c they need to have the ff_* prefix added to them.

Other than that it seems fine. Definitely needs to be squashed before
committing though.

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

Reply via email to