[vlc-devel] [PATCH] fix wrong EBML element checking before a cast

Jean-Baptiste Kempf jb at videolan.org
Fri Feb 20 14:56:34 CET 2015


Applied.

On 20 Feb, Steve Lhomme wrote :
> Fix a silly typo and loads Tags the first time we see them, during the
> Preload phase.
> 
> ---
> modules/demux/mkv/matroska_segment.cpp | 6 +++---
> 1 file changed, 3 insertions(+), 3 deletions(-)
> 
> diff --git a/modules/demux/mkv/matroska_segment.cpp
> b/modules/demux/mkv/matroska_segment.cpp
> index 904a12f..eac5a54 100644
> --- a/modules/demux/mkv/matroska_segment.cpp
> +++ b/modules/demux/mkv/matroska_segment.cpp
> @@ -726,11 +726,11 @@ bool matroska_segment_c::Preload( )
> ParseChapters( static_cast( el ) );
> i_chapters_position = (int64_t) es.I_O().getFilePointer();
> }
> - else if( MKV_IS_ID( el, KaxTag ) )
> + else if( MKV_IS_ID( el, KaxTags ) )
> {
> msg_Dbg( &sys.demuxer, "| + Tags" );
> - /*FIXME if( i_tags_position < 0)
> - LoadTags( static_cast( el ) );*/
> + if( i_tags_position < 0)
> + LoadTags( static_cast( el ) );
> i_tags_position = (int64_t) es.I_O().getFilePointer();
> }
> else if( MKV_IS_ID( el, EbmlVoid ) )
> -- 
> 1.9.5.msysgit.0
> 

> _______________________________________________
> vlc-devel mailing list
> To unsubscribe or modify your subscription options:
> https://mailman.videolan.org/listinfo/vlc-devel


-- 
With my kindest regards,

-- 
Jean-Baptiste Kempf
http://www.jbkempf.com/ - +33 672 704 734
Sent from my Electronic Device



More information about the vlc-devel mailing list