]> git.sesse.net Git - vlc/commitdiff
transcode: add comments on why we check frame dropping on 2 places
authorIlkka Ollakka <ileoo@videolan.org>
Sat, 1 Feb 2014 10:39:04 +0000 (12:39 +0200)
committerIlkka Ollakka <ileoo@videolan.org>
Sun, 2 Feb 2014 13:58:20 +0000 (15:58 +0200)
modules/stream_out/transcode/video.c

index 12c974e5b85c1bf7d1e288eeb27adecc8796168c..0c811cae006732825ec8a55a75ee5f7250b5bf34 100644 (file)
@@ -670,7 +670,10 @@ static void OutputFrame( sout_stream_sys_t *p_sys, picture_t *p_pic, sout_stream
     /* If input pts + input_frame_interval is lower than next_output_pts - output_frame_interval
      * Then the future input frame should fit better and we can drop this one 
      *
-     * Duplication need is checked in OutputFrame */
+     * We check it here also because we can have case that video filters outputs multiple
+     * pictures but we don't need to use them all, for example yadif2x and outputting to some
+     * different fps value
+     */
     if( ( p_pic->date + (mtime_t)id->i_input_frame_interval ) <
         ( date_Get( &id->next_output_pts ) ) )
     {
@@ -934,6 +937,9 @@ int transcode_video_process( sout_stream_t *p_stream, sout_stream_id_t *id,
             /* If input pts + input_frame_interval is lower than next_output_pts - output_frame_interval
              * Then the future input frame should fit better and we can drop this one 
              *
+             * We check this here as we don't need to run video filter at all for pictures
+             * we are going to drop anyway
+             *
              * Duplication need is checked in OutputFrame */
             if( ( p_pic->date + (mtime_t)id->i_input_frame_interval ) <
                 ( date_Get( &id->next_output_pts ) ) )