[FFmpeg-cvslog] r13242 - trunk/libavformat/matroskadec.c

Michael Niedermayer michaelni
Fri May 23 01:46:17 CEST 2008


On Fri, May 23, 2008 at 01:04:02AM +0200, aurel wrote:
> Author: aurel
> Date: Fri May 23 01:04:02 2008
> New Revision: 13242
> 
> Log:
> matroskadec: allows inserting chapters with unspecified end
> 
> Modified:
>    trunk/libavformat/matroskadec.c
> 
> Modified: trunk/libavformat/matroskadec.c
> ==============================================================================
> --- trunk/libavformat/matroskadec.c	(original)
> +++ trunk/libavformat/matroskadec.c	Fri May 23 01:04:02 2008
> @@ -2250,8 +2250,12 @@ matroska_parse_chapters(AVFormatContext 
>                          }
>                      }
>  
> -                    if(start != AV_NOPTS_VALUE && end != AV_NOPTS_VALUE)
> -                        res = ff_new_chapter(s, start * AV_TIME_BASE / 1000000000 , end * AV_TIME_BASE / 1000000000, title ? title : "(unnamed)");
> +                    if (start != AV_NOPTS_VALUE) {
> +                        start = start * AV_TIME_BASE / 1000000000;
> +                        if (end != AV_NOPTS_VALUE)
> +                            end = end * AV_TIME_BASE / 1000000000;
> +                        res = ff_new_chapter(s, start, end, title ? title : "(unnamed)");
> +                    }
>                      av_free(title);

What does a chapter with no end mean? Can there be more than one?
Must it be the last?
Does it end with the start of the next? If so we need code to fix up
these missing ends.

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

No snowflake in an avalanche ever feels responsible. -- Voltaire
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: Digital signature
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-cvslog/attachments/20080523/3a6baef2/attachment.pgp>



More information about the ffmpeg-cvslog mailing list