[FFmpeg-cvslog] oggdec: simplify start time calculation code.

Reimar Döffinger git at videolan.org
Sun Aug 5 19:48:17 CEST 2012


ffmpeg | branch: master | Reimar Döffinger <Reimar.Doeffinger at gmx.de> | Sun Aug  5 09:41:20 2012 +0200| [a4163b2d6583396845e06b1b976c3a981eb8cb1c] | committer: Reimar Döffinger

oggdec: simplify start time calculation code.

Also slightly more correct behaviour in case streams_left for
some reason is 0 from the start.

Signed-off-by: Reimar Döffinger <Reimar.Doeffinger at gmx.de>

> http://git.videolan.org/gitweb.cgi/ffmpeg.git/?a=commit;h=a4163b2d6583396845e06b1b976c3a981eb8cb1c
---

 libavformat/oggdec.c |   10 ++++------
 1 file changed, 4 insertions(+), 6 deletions(-)

diff --git a/libavformat/oggdec.c b/libavformat/oggdec.c
index 0a46501..cb149ed 100644
--- a/libavformat/oggdec.c
+++ b/libavformat/oggdec.c
@@ -526,9 +526,10 @@ static int ogg_get_length(AVFormatContext *s)
     avio_seek (s->pb, s->data_offset, SEEK_SET);
     ogg_reset(s);
     i = -1;
-    while (!ogg_packet(s, &i, NULL, NULL, NULL)) {
-        if(i>=0) {
-        int64_t pts = ogg_calc_pts(s, i, NULL);
+    while (streams_left > 0 && !ogg_packet(s, &i, NULL, NULL, NULL)) {
+        int64_t pts;
+        if (i < 0) continue;
+        pts = ogg_calc_pts(s, i, NULL);
         if (pts != AV_NOPTS_VALUE && s->streams[i]->start_time == AV_NOPTS_VALUE && !ogg->streams[i].got_start){
             s->streams[i]->duration -= pts;
             ogg->streams[i].got_start= 1;
@@ -537,9 +538,6 @@ static int ogg_get_length(AVFormatContext *s)
             ogg->streams[i].got_start= 1;
             streams_left--;
         }
-        }
-            if(streams_left<=0)
-                break;
     }
     ogg_restore (s, 0);
 



More information about the ffmpeg-cvslog mailing list