Re: [FFmpeg-devel] [PATCH] Cleaning existing FFV1 tests

2015-08-31 Thread Michael Niedermayer
On Mon, Aug 31, 2015 at 03:17:22AM +0200, Peter B. wrote: > Hi. > > I've now cleaned up a few things in the current FFV1 tests: > - Changed "ffv1.X" to "ffv1-vX", removing necessity for vcodec argument. > - Added linebreaks to FFV1 FATE tests for better readability. > - Renamed pix_fmts wording in

Re: [FFmpeg-devel] [PATCH] Cleaning existing FFV1 tests

2015-08-31 Thread Carl Eugen Hoyos
Hendrik Leppkes gmail.com> writes: > What should not be changed is the samples inside the > FATE suite - if a change is needed, there should be > a new sample with a new name - which is probably what > Carl refers to. Yes. > But since these seem to be vsynth tests, no external > samples fro

Re: [FFmpeg-devel] [PATCH] Cleaning existing FFV1 tests

2015-08-31 Thread Peter B.
Am Mo, 31.08.2015, 11:22 schrieb Hendrik Leppkes: > On Mon, Aug 31, 2015 at 11:13 AM, Peter B. wrote: >> >> In my case, there are files in "tests/ref/vsynth" that contain the >> expected reference results from tests. >> >> If I only provide the changed the FATE makefiles, like "vcodec.mak", who >>

Re: [FFmpeg-devel] [PATCH] Cleaning existing FFV1 tests

2015-08-31 Thread Hendrik Leppkes
On Mon, Aug 31, 2015 at 11:13 AM, Peter B. wrote: > > > Am Mo, 31.08.2015, 08:31 schrieb Carl Eugen Hoyos: >> Peter B. das-werkstatt.com> writes: >> >>> Replaced the reference files accordingly >> >> This cannot be done, we want to run fate (also) >> with old sources. > > Hm. Now I'm confused...

Re: [FFmpeg-devel] [PATCH] Cleaning existing FFV1 tests

2015-08-31 Thread Peter B.
Am Mo, 31.08.2015, 08:31 schrieb Carl Eugen Hoyos: > Peter B. das-werkstatt.com> writes: > >> Replaced the reference files accordingly > > This cannot be done, we want to run fate (also) > with old sources. Hm. Now I'm confused... I expected old sources to use old tests? In my case, there are

Re: [FFmpeg-devel] [PATCH] Cleaning existing FFV1 tests

2015-08-30 Thread Carl Eugen Hoyos
Peter B. das-werkstatt.com> writes: > Replaced the reference files accordingly This cannot be done, we want to run fate (also) with old sources. Carl Eugen ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmp

Re: [FFmpeg-devel] [PATCH] Cleaning existing FFV1 tests

2015-08-30 Thread Peter B.
Hi. I've now cleaned up a few things in the current FFV1 tests: - Changed "ffv1.X" to "ffv1-vX", removing necessity for vcodec argument. - Added linebreaks to FFV1 FATE tests for better readability. - Renamed pix_fmts wording in test target name to match pix_fmt parameter. Replaced the reference