[FFmpeg-devel] [PATCH v9] lavc/rawdec: Remove monowhite switching code for 1 bpp AVI without a palette

Mats Peterson matsp888 at yahoo.com
Fri Feb 12 12:53:46 CET 2016


On 02/12/2016 12:49 PM, Michael Niedermayer wrote:
> please split the patch so that each patch is selfcontained and
> does not contain independant changes or bugfixes.
> see "git rebase -i" and 'git add -p' and "git commit --amend" for
> tools to conveniently seperate changes into commits
>
> It is important for git history, future bisect/debug and even
> yourself understanding changes in the future to cleanly split
> commits.
>
>

If it was that easy. Everything is dependent on each other in this 
patch, in my book. If you have a better way to split it, please go 
ahead. I know the debug code is superfluous, though.

Mats



More information about the ffmpeg-devel mailing list