[vlc-commits] MKV: Chapter timestamps are already set, don't overwrite them

[Steve Lhomme]

vlc | branch: master | Steve Lhomme <robUx4 at videolabs.io> | Mon Feb 23 17:05:37 2015 +0000| [95cfad6a4052a4e7361eb451b513c9c90043e443] | committer: Jean-Baptiste Kempf

MKV: Chapter timestamps are already set, don't overwrite them

It is mandatory in the specs. The chapters are not supposed to be in chronological order.

Fixes #13773

Signed-off-by: Jean-Baptiste Kempf <jb at videolan.org>

> http://git.videolan.org/gitweb.cgi/vlc.git/?a=commit;h=95cfad6a4052a4e7361eb451b513c9c90043e443
---

 modules/demux/mkv/virtual_segment.cpp |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/modules/demux/mkv/virtual_segment.cpp b/modules/demux/mkv/virtual_segment.cpp
index 922b686..a41abb8 100644
--- a/modules/demux/mkv/virtual_segment.cpp
+++ b/modules/demux/mkv/virtual_segment.cpp
@@ -226,7 +226,7 @@ void virtual_edition_c::retimeSubChapters( virtual_chapter_c * p_vchap )
     for( size_t i = p_vchap->sub_chapters.size(); i-- > 0; )
     {
         virtual_chapter_c * p_vsubchap = p_vchap->sub_chapters[i];
-        p_vsubchap->i_virtual_start_time += p_vchap->i_virtual_start_time;
+        //p_vsubchap->i_virtual_start_time += p_vchap->i_virtual_start_time;
 
         /*FIXME we artificially extend stop time if they were there before...*/
         /* Just for comfort*/