[FFmpeg-devel] [PATCH] s302menc: Add S302M encoder.
Reimar.Doeffinger at gmx.de
Wed Apr 11 21:54:09 CEST 2012
On Wed, Apr 11, 2012 at 09:15:59AM -0700, Thierry Foucu wrote:
> regarding regression test: Do you have some example how I can look at to
> make one?
> Will it be ok if the regression is part of another patch?
Experience says that not having one will not hinder the patch.
However that has also caused the current state where we have barely
60% test coverage and a lot of the untested stuff is broken
in some way.
So simply in the interest of making sure your code will stay usable
as long as possible consider doing it right now.
I have to admit that adding a codec-regression one is a bit tricky,
to "enable" a new test you need to create a file in
tests/ref/acodec/ with a name that fits the "if $do_" check in
codec-regressions.sh and then run configure again.
(Or if you like me think that takes too much time, just add
it manually to ACODEC_TESTS in config.mak).
More information about the ffmpeg-devel