[FFmpeg-devel] [PATCH] webm_chunk: Set pts precision to milliseconds
Vignesh Venkatasubramanian
vigneshv at google.com
Thu Aug 25 21:46:15 EEST 2016
On Mon, Aug 22, 2016 at 5:13 PM, Vignesh Venkatasubramanian
<vigneshv at google.com> wrote:
> Milliseconds is the de-factor precision for timestamps in
> Matroska/WebM media.
>
> Signed-off-by: Vignesh Venkatasubramanian <vigneshv at google.com>
> ---
> libavformat/webm_chunk.c | 5 +++++
> 1 file changed, 5 insertions(+)
>
> diff --git a/libavformat/webm_chunk.c b/libavformat/webm_chunk.c
> index 9db4fab..44a1695 100644
> --- a/libavformat/webm_chunk.c
> +++ b/libavformat/webm_chunk.c
> @@ -110,6 +110,7 @@ static int webm_chunk_write_header(AVFormatContext *s)
> WebMChunkContext *wc = s->priv_data;
> AVFormatContext *oc = NULL;
> int ret;
> + int i;
>
> // DASH Streams can only have either one track per file.
> if (s->nb_streams != 1) { return AVERROR_INVALIDDATA; }
> @@ -135,6 +136,10 @@ static int webm_chunk_write_header(AVFormatContext *s)
> if (ret < 0)
> return ret;
> ff_format_io_close(s, &oc->pb);
> + for (i = 0; i < s->nb_streams; i++) {
> + // ms precision is the de-facto standard timescale for mkv files.
> + avpriv_set_pts_info(s->streams[i], 64, 1, 1000);
> + }
> return 0;
> }
>
> --
> 2.8.0.rc3.226.g39d4020
>
Can somebody take a look at this please? This fixes timestamp writing
issues in the webm chunk muxer. Thanks!
--
Vignesh
More information about the ffmpeg-devel
mailing list