[FFmpeg-devel] [PATCH] Make RTP work with IPv6 enabled v.2

Ronald S. Bultje rsbultje
Mon Oct 29 13:28:20 CET 2007


Hi Nicolas,

On 10/29/07, Nicolas George <nicolas.george at normalesup.org> wrote:
>
> By the way, I am a little unsure about a point of policy: adding or
> removing
> a test on a few lines of code implies to add or remove one level of
> indentation on the affected lines. What is the preferred way to submit
> such
> a patch? Keeping it as one patch mixes cosmetic and functional, but
> splitting it means one revision with incorrect indentation. Common sense
> tells me to keep one patch if the number of lines is very low, and to
> split
> if it is high. Is there an official statement about it?


I think it's always been that they should unconditionally ( :-) ) be
separated, hence my re-submission. Thanks to you for finding and fixing the
bug, of course.

Ronald




More information about the ffmpeg-devel mailing list