"Dan Kegel" <[EMAIL PROTECTED]> wrote:

>> That's an optional file that could be dropped in into the tests
>> directory so that the test parses some real AVI file along with
>> the (synthesized) data existing in the test. That could be an AVI
>> from any source, there is no particular requirements. The only
>> purpose of referencing msrle.avi is to (optionally) extend the test
>> coverage.
> 
> OK... any reason we don't check in a real .avi into git?

The test already has the data resembling an AVI file, and checking
in a real AVI into git would actually reduce the coverage IMO, not
extend it (anyone interested should be able to drop its own AVI).
And 'optional' really means optional.

-- 
Dmitry.


Reply via email to