[FFmpeg-devel] [PATCH 2/2] avformat/framehash: Add more information to the output

Hendrik Leppkes h.leppkes at gmail.com
Tue Apr 12 17:38:15 CEST 2016

On Tue, Apr 12, 2016 at 4:56 PM, Ronald S. Bultje <rsbultje at gmail.com> wrote:
> Hi James,
> On Sun, Apr 10, 2016 at 5:22 PM, James Almer <jamrial at gmail.com> wrote:
>> From: Michael Niedermayer <michael at niedermayer.cc>
>> Signed-off-by: Michael Niedermayer <michael at niedermayer.cc>
>> Signed-off-by: James Almer <jamrial at gmail.com>
>> ---
>> The differences with Michael's original patch are that this one is updated
>> to use codecpar, and adds the new output as framehash version 2 but doesn't
>> make it the default until further additions are made, like porting
>> framecrc's
>> extradata and sidedata checksum.
> Does this one support frame size notifications? E.g. if vp8/h264/vp9
> frame-size-change fate tests actually change size, does the md5 pick up on
> this in the way you'd expect? Right now we rescale these fate tests with
> swscale which is awful.

framehash uses rawvideo, which in itself doesn't support frame size
changes, since it has no concept to change any metadata.
Maybe this could be done using the new wrapped_avframe pseudo-codec,
which carries a full AVFrame including size information.

- Hendrik

More information about the ffmpeg-devel mailing list