Pass time_base as argument to new_chapter() as well.
authorMichael Niedermayer <michaelni@gmx.at>
Fri, 23 May 2008 18:15:13 +0000 (18:15 +0000)
committerMichael Niedermayer <michaelni@gmx.at>
Fri, 23 May 2008 18:15:13 +0000 (18:15 +0000)
This fixes the wrong timebase the matroska demuxer had after my previous commits.
Maybe we should reduce new_chapter() to just (AVFormatContext, int id) ?

Originally committed as revision 13266 to svn://svn.ffmpeg.org/ffmpeg/trunk

libavformat/avformat.h
libavformat/matroskadec.c
libavformat/nutdec.c
libavformat/utils.c

index fd49442..f8f35ac 100644 (file)
@@ -776,13 +776,13 @@ AVProgram *av_new_program(AVFormatContext *s, int id);
  *
  * @param s media file handle
  * @param id unique id for this chapter
  *
  * @param s media file handle
  * @param id unique id for this chapter
- * @param start chapter start time in AV_TIME_BASE units
- * @param end chapter end time in AV_TIME_BASE units
+ * @param start chapter start time in time_base units
+ * @param end chapter end time in time_base units
  * @param title chapter title
  *
  * @return AVChapter or NULL if error.
  */
  * @param title chapter title
  *
  * @return AVChapter or NULL if error.
  */
-AVChapter *ff_new_chapter(AVFormatContext *s, int id, int64_t start, int64_t end, const char *title);
+AVChapter *ff_new_chapter(AVFormatContext *s, int id, AVRational time_base, int64_t start, int64_t end, const char *title);
 
 /**
  * Set the pts for a given stream.
 
 /**
  * Set the pts for a given stream.
index 7cbf705..6f6caa1 100644 (file)
@@ -2254,10 +2254,7 @@ matroska_parse_chapters(AVFormatContext *s)
                     }
 
                     if (start != AV_NOPTS_VALUE && uid != -1) {
                     }
 
                     if (start != AV_NOPTS_VALUE && uid != -1) {
-                        start = start * AV_TIME_BASE / 1000000000;
-                        if (end != AV_NOPTS_VALUE)
-                            end = end * AV_TIME_BASE / 1000000000;
-                        if(!ff_new_chapter(s, uid, start, end, title))
+                        if(!ff_new_chapter(s, uid, (AVRational){1, 1000000000}, start, end, title))
                             res= AVERROR(ENOMEM);
                     }
                     av_free(title);
                             res= AVERROR(ENOMEM);
                     }
                     av_free(title);
index 7259a36..0655d0f 100644 (file)
@@ -406,8 +406,9 @@ static int decode_info_header(NUTContext *nut){
 
     if(chapter_id && !stream_id_plus1){
         int64_t start= chapter_start / nut->time_base_count;
 
     if(chapter_id && !stream_id_plus1){
         int64_t start= chapter_start / nut->time_base_count;
-        chapter= ff_new_chapter(s, chapter_id, start, start + chapter_len, NULL);
-        chapter->time_base= nut->time_base[chapter_start % nut->time_base_count];
+        chapter= ff_new_chapter(s, chapter_id,
+                                nut->time_base[chapter_start % nut->time_base_count],
+                                start, start + chapter_len, NULL);
     }
 
     for(i=0; i<count; i++){
     }
 
     for(i=0; i<count; i++){
index e37577d..7faedcc 100644 (file)
@@ -2246,7 +2246,7 @@ void av_set_program_name(AVProgram *program, char *provider_name, char *name)
     }
 }
 
     }
 }
 
-AVChapter *ff_new_chapter(AVFormatContext *s, int id, int64_t start, int64_t end, const char *title)
+AVChapter *ff_new_chapter(AVFormatContext *s, int id, AVRational time_base, int64_t start, int64_t end, const char *title)
 {
     AVChapter *chapter = NULL;
     int i;
 {
     AVChapter *chapter = NULL;
     int i;
@@ -2265,6 +2265,7 @@ AVChapter *ff_new_chapter(AVFormatContext *s, int id, int64_t start, int64_t end
         av_free(chapter->title);
     chapter->title = av_strdup(title);
     chapter->id    = id;
         av_free(chapter->title);
     chapter->title = av_strdup(title);
     chapter->id    = id;
+    chapter->time_base= time_base;
     chapter->start = start;
     chapter->end   = end;
 
     chapter->start = start;
     chapter->end   = end;