[FFmpeg-devel] [PATCHv3 1/4] fate: add concat demuxer tests

Marton Balint cus at passwd.hu
Thu Nov 5 22:32:52 CET 2015


On Thu, 5 Nov 2015, Clément Bœsch wrote:

> On Wed, Nov 04, 2015 at 03:33:16PM +0100, Nicolas George wrote:
>> Le quartidi 14 brumaire, an CCXXIV, Marton Balint a écrit :
>>> +      run ffprobe${PROGSUF} -show_streams -show_packets -v 0 -fflags keepside -safe 0 $extra_args $concatfile > $packetfile
>>
>> Maybe -of compact, that makes the reference files quite smaller.
>
> I'd also suggest to use -show_entries packets=... with a few fields. It
> will avoid updating this reference file every time we add new fields.

Adding new fields to AVPacket is not that common, and I don't think it is 
a problem that you have to update the fate tests. I'd rather not limit the 
fields, a newly added field to AVPacket can be as relevant as an existing 
one.

Regards,
Marton


More information about the ffmpeg-devel mailing list