[FFmpeg-devel] [PATCH] webm_chunk: Set pts precision to milliseconds

[Vignesh Venkatasubramanian]

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