[FFmpeg-devel] [PATCH 5/5] tests: add coverage for libavfilter's format negotiation

Nicolas George george at nsup.org
Thu Mar 3 20:37:32 EET 2022


James Almer (12022-03-03):
> No need to add a fate-libavfilter-negotiation target until there are more
> such tests.

Thanks for the review. I know these targets are not necessary, but I
want them. I do intend to write more tests:

>> This is not to be the only test, I intend to cover all the logic in
>> pick_format() and the logic in swap_*().

And having submodule-specific fate targets is convenient even if there
are only one test.

Please read the code assuming there are several tests, including tests
using aresample rather than scale.

Regards,

-- 
  Nicolas George
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 833 bytes
Desc: not available
URL: <https://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20220303/ae823bfd/attachment.sig>


More information about the ffmpeg-devel mailing list