On Thu, 29 May 2014 12:25:49 +0300, Martin Storsjö <[email protected]> wrote:
> While it strictly isn't necessary to copy the time base (since
> any use of it is scaled in ff_write_chained), it still is better
> to signal the actual time base to the caller, avoiding one
> unnecessary rescaling. This also lets the caller know what the
> actual internal time base is, in case that is useful info
> for some caller.
> 
> This reverts commit 397ffde115f4e0482a007b672f40457596cebfc4.
> ---
>  libavformat/rtpenc_chain.c | 3 +++
>  1 file changed, 3 insertions(+)
> 
> diff --git a/libavformat/rtpenc_chain.c b/libavformat/rtpenc_chain.c
> index 10c4020..3806254 100644
> --- a/libavformat/rtpenc_chain.c
> +++ b/libavformat/rtpenc_chain.c
> @@ -96,6 +96,9 @@ int ff_rtp_chain_mux_open(AVFormatContext **out, 
> AVFormatContext *s,
>          return ret;
>      }
>  
> +    /* Copy the RTP AVStream timebase back to the original AVStream */
> +    st->time_base = rtpctx->streams[0]->time_base;
> +
>      *out = rtpctx;
>      return 0;
>  
> -- 
> 1.8.5.2 (Apple Git-48)
> 

Fine with me as well, as already discussed on IRC.

-- 
Anton Khirnov
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to