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

Marton Balint cus at passwd.hu
Sat Jul 16 02:53:26 EEST 2016

On Wed, 13 Jul 2016, Jan Sebechlebsky wrote:

> 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.

Thanks, applied.


More information about the ffmpeg-devel mailing list