[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