On Sun, Mar 09, 2014 at 10:21:36AM +0100, Vittorio Giovara wrote:
> ---
> What's a decoder without a fate entry?
> I've chosen the smallest sample available and set to decoding 300 frames only.
> I can reduce/increase the number of frames and/or pick another sample if 
> requested.
> Cheers,
>     Vittorio
>  tests/fate/vpx.mak |   3 +
>  tests/ref/fate/vp7 | 301 
> +++++++++++++++++++++++++++++++++++++++++++++++++++++
>  2 files changed, 304 insertions(+)
>  create mode 100644 tests/ref/fate/vp7

You are adding a test, not a sample.

300 frames sounds like a lot to me.  Do we really need that many to
achieve high coverage?

The patch itself LGTM otherwise.

Diego
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to