[FFmpeg-devel] [PATCH] fate: test flac LPC encoder
James Darnley
james.darnley at gmail.com
Thu Aug 14 20:49:14 CEST 2014
After further searching and understanding the makefile I see that the
lpc code is tested by the complete fate sample suite. This patch is
unnecessary.
Anyway to answer the questions...
On 2014-08-14 18:14, Christophe Gisquet wrote:
> Hi,
>
> 2014-08-14 18:08 GMT+02:00 James Darnley <james.darnley at gmail.com>:
>> Minimum 1, maximum 8. Should I find out what order(s) are actually chosen?
>
> The encoder algorithm might change and make that information useless,
That would mean the test result needs changing too.
> but that could be useful. I fear that only checking which orders it
> *tests* is insufficient: if the dsp function is broken for some
> orders, it may just lead to not selecting those orders and not change
> the output.
It may be possible that it could be broken in this manner but it sounds,
to me, very unlikely.
FYI: 1, 4, 6, 7, 8 are all used at least once.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 618 bytes
Desc: OpenPGP digital signature
URL: <https://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20140814/f5422b08/attachment.asc>
More information about the ffmpeg-devel
mailing list