[FFmpeg-devel] [PATCH] tests: Download lena.pnm if its missing from the source tree

Andreas Cadhalpun andreas.cadhalpun at googlemail.com
Sun Aug 31 14:35:40 CEST 2014


On 28.08.2014 20:06, Nicolas George wrote:
> Le primidi 11 fructidor, an CCXXII, Andreas Cadhalpun a écrit :
>> I like this colour of the shed. ;)
>>   * move lena.pnm to the FATE samples
>>   * add e.g. mona_lisa.pnm and duplicate the vsynth2 tests (as vsynth4?)
>>
>> Has someone a problem with this solution?
>
> I find it very good.
>
> Just a small nitpick: does the image have to be 256×256? If not, I suggest
> to make mona_lisa.pnm 256×384 or 192×288, to avoid cropping the original.

Yes, 256×384 fits better.
Attached is a patch on top of the one removing tests/lena.pnm, adding 
tests/mona_lisa.pnm and vsynth4 tests.

Best regards,
Andreas
-------------- next part --------------
A non-text attachment was scrubbed...
Name: duplicate-vsynth2-tests-as-vsynth4.patch
Type: text/x-diff
Size: 502539 bytes
Desc: not available
URL: <https://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20140831/aed27060/attachment.bin>


More information about the ffmpeg-devel mailing list