[FFmpeg-devel] [PATCH] swscale-test: add md5 output
Tue Aug 3 01:29:59 CEST 2010
On Sat, Jul 31, 2010 at 7:47 PM, Vitor Sessak <vitor1001 at gmail.com> wrote:
> On 08/01/2010 12:31 AM, Ramiro Polla wrote:
>> On Sat, Jul 31, 2010 at 7:27 PM, Vitor Sessak<vitor1001 at gmail.com> ?wrote:
>>> On 07/31/2010 11:10 PM, Ramiro Polla wrote:
>>>> On Sun, Jul 25, 2010 at 9:54 PM, Michael Niedermayer<michaelni at gmx.at>
>>>>> SSE of 123 does not implicate that there is no change to a previous SSE
>>>>> of 123
>>>>> and a change of CRC does tell nothing of the SSE
>>>> For regression tests we want a checksum to be bitexact (I'll send a
>>>> patch to add the bitexact flag to sws_flags later), and in this case
>>>> the SSE doesn't matter.
>>> It does, because every time there is a patch that is expected to change
>>> CRC (and thus update the reference output of the tests) it is easy to
>>> if it improves or reduce the PSNR by just looking at the diff of the
>>> reference file. That's the reason why "make test" prints also the
>> Hm, I was hoping that whenever the checksum changed, the dev would
>> himself run swscale-test -sse...
> It is always clearer to see the stddev before and after the patch right
> there in the diff in case the reviewer want to know...
>> So do you think it'd be best to always print out both checksum and sse?
> I think it is worth the few extra bytes per test.
New patch attached.
-------------- next part --------------
A non-text attachment was scrubbed...
Size: 1308 bytes
Desc: not available
More information about the ffmpeg-devel