[FFmpeg-devel] [PATCH 2/7] avformat/tee: Use ff_stream_encode_params_copy()

[Jan Sebechlebsky]

On 07/13/2016 12:56 PM, Jan Sebechlebsky wrote:
>
>
> On 07/12/2016 10:23 AM, Nicolas George wrote:
>> Le quartidi 24 messidor, an CCXXIV, Jan Sebechlebsky a écrit :
>>> I think it is used in decoding only - the documentation of AVStream 
>>> mentions
>>> only decoding and I also tried to search references to that field if 
>>> it is
>>> not used in some of the muxers. If you think it should be copied too 
>>> I will
>>> add it.
>> If you have checked it was never used in muxers, then ok. But you may 
>> want
>> to have a look at the "MKV Header: Writing duration early" mail that 
>> arrived
>> a few minutes ago.
>>
>> Regards,
>>
> I think we can safely omit it, it's not used in any current muxers.
> Soft Works contributors may use duration field in muxing context, I 
> think they (or we) can then add copying of this field to 
> ff_stream_encode_params_copy with their patch . The documentation of 
> the field should be then modified as well.
>
> Regards,
> Jan
Resending this to mailing list - sorry I've accidentally replied 
privately again.

Regards,
Jan