On Wed, Nov 23, 2016 at 4:08 PM, Michael Niedermayer
<mich...@niedermayer.cc> wrote:
>
> please read
> http://ffmpeg.org/developer.html
>
> for example it says:
> "Do not commit unrelated changes together."
>
> you combined commits which were split before

Those changes are closely related. But I will merge those six commits
one at a time.

> "Testing must be adequate but not excessive.
>  If it works for you, others, and passes FATE then it should be OK to commit 
> it, provided it fits the other committing criteria. You should not worry 
> about over-testing things. If your code has problems (portability, triggers 
> compiler bugs, unusual environment etc) they will be reported and eventually 
> fixed. "
>
> this fails fate

Sorry about that! I did run "make fate", but I must have forgotten to
check the test results.

I assume I should use the standard FFmpeg license header in the new .h files?

Thanks,
Wan-Teh Chang
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Reply via email to