[vlc-devel] [PATCH 11/30] mkv: removed unnecessary call to es_out_Control

Filip Roséen filip at videolabs.io
Mon May 9 14:53:05 CEST 2016


We are always calling matroska_segment_c::Seek after this function
(matroska_segment_c::Select) is called, as such there is absolutely no
need for us to explicitly invoke ES_OUT_SET_NEXT_DISPLAY_TIME (since
this will happen implicitly later).
---
 modules/demux/mkv/matroska_segment.cpp | 7 -------
 1 file changed, 7 deletions(-)

diff --git a/modules/demux/mkv/matroska_segment.cpp b/modules/demux/mkv/matroska_segment.cpp
index 017db2c..db45b04 100644
--- a/modules/demux/mkv/matroska_segment.cpp
+++ b/modules/demux/mkv/matroska_segment.cpp
@@ -956,13 +956,6 @@ bool matroska_segment_c::Select( mtime_t i_mk_start_time )
             es_out_Control( sys.demuxer.out, ES_OUT_SET_ES_DEFAULT, track.p_es );
         }
     }
-    es_out_Control( sys.demuxer.out, ES_OUT_SET_NEXT_DISPLAY_TIME, i_mk_start_time );
-
-    sys.i_start_pts = i_mk_start_time + VLC_TS_0;
-    // reset the stream reading to the first cluster of the segment used
-    es.I_O().setFilePointer( 0 /* previously i_start_pos */ );
-
-    ep->reconstruct( &es, segment, &sys.demuxer );
 
     return true;
 }
-- 
2.8.2



More information about the vlc-devel mailing list