This fixes the jerky video from vob files. However, I am convinced that this
authorPhilip Gladstone <philipjsg@users.sourceforge.net>
Wed, 25 Dec 2002 04:03:30 +0000 (04:03 +0000)
committerPhilip Gladstone <philipjsg@users.sourceforge.net>
Wed, 25 Dec 2002 04:03:30 +0000 (04:03 +0000)
is the wrong solution to the problem. In particular, if you specify an output
frame rate, then it doesn't work correctly. My feeling is that the pts value
ought to be interpolated between those frames where the pts is actually
known. Maybe someone else could do that (change the line that I added,
setting a value equal to AV_NOPTS_VALUE).
Happy Christmas.

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

ffmpeg.c

index e0c7cb1..4042bb7 100644 (file)
--- a/ffmpeg.c
+++ b/ffmpeg.c
@@ -555,16 +555,18 @@ static void do_video_out(AVFormatContext *s,
 
         if (ost->sync_ipts != AV_NOPTS_VALUE) {
             vdelta = (double)(ost->st->pts.val) * s->pts_num / s->pts_den - (ost->sync_ipts - ost->sync_ipts_offset);
 
         if (ost->sync_ipts != AV_NOPTS_VALUE) {
             vdelta = (double)(ost->st->pts.val) * s->pts_num / s->pts_den - (ost->sync_ipts - ost->sync_ipts_offset);
-            if (vdelta < 100 && vdelta > -100) {
+            if (vdelta < 100 && vdelta > -100 && ost->sync_ipts_offset) {
                 if (vdelta < -AV_DELAY_MAX)
                     nb_frames = 2;
                 else if (vdelta > AV_DELAY_MAX)
                     nb_frames = 0;
             } else {
                 ost->sync_ipts_offset -= vdelta;
                 if (vdelta < -AV_DELAY_MAX)
                     nb_frames = 2;
                 else if (vdelta > AV_DELAY_MAX)
                     nb_frames = 0;
             } else {
                 ost->sync_ipts_offset -= vdelta;
+                if (!ost->sync_ipts_offset)
+                    ost->sync_ipts_offset = 0.000001; /* one microsecond */
             }
 
             }
 
-#if 0
+#if defined(PJSG)
             {
                 static char *action[] = { "drop frame", "copy frame", "dup frame" };
                 printf("Input PTS %12.6f, output PTS %12.6f: %s\n",
             {
                 static char *action[] = { "drop frame", "copy frame", "dup frame" };
                 printf("Input PTS %12.6f, output PTS %12.6f: %s\n",
@@ -1369,6 +1371,7 @@ static int av_encode(AVFormatContext **output_files,
                         //printf("ipts=%lld sync_ipts=%f sync_opts=%lld pts.val=%lld pkt.pts=%lld\n", ipts, ost->sync_ipts, ost->sync_opts, ost->st->pts.val, pkt.pts); 
                     } else {
                         //printf("pts.val=%lld\n", ost->st->pts.val); 
                         //printf("ipts=%lld sync_ipts=%f sync_opts=%lld pts.val=%lld pkt.pts=%lld\n", ipts, ost->sync_ipts, ost->sync_opts, ost->st->pts.val, pkt.pts); 
                     } else {
                         //printf("pts.val=%lld\n", ost->st->pts.val); 
+                        ost->sync_ipts = AV_NOPTS_VALUE;
                     }
 
                     if (ost->encoding_needed) {
                     }
 
                     if (ost->encoding_needed) {