Commit eed05ce7 authored by Ilkka Ollakka's avatar Ilkka Ollakka

transcode: add comments on why we check frame dropping on 2 places

parent dee8cee6
...@@ -670,7 +670,10 @@ static void OutputFrame( sout_stream_sys_t *p_sys, picture_t *p_pic, sout_stream ...@@ -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 /* 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 * 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 ) < if( ( p_pic->date + (mtime_t)id->i_input_frame_interval ) <
( date_Get( &id->next_output_pts ) ) ) ( date_Get( &id->next_output_pts ) ) )
{ {
...@@ -934,6 +937,9 @@ int transcode_video_process( sout_stream_t *p_stream, sout_stream_id_t *id, ...@@ -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 /* 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 * 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 */ * Duplication need is checked in OutputFrame */
if( ( p_pic->date + (mtime_t)id->i_input_frame_interval ) < if( ( p_pic->date + (mtime_t)id->i_input_frame_interval ) <
( date_Get( &id->next_output_pts ) ) ) ( date_Get( &id->next_output_pts ) ) )
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment