[vlc-commits] demux: mkv: Remove unnecessary throw

Soomin Lee git at videolan.org
Wed Nov 28 10:24:07 CET 2018


vlc | branch: master | Soomin Lee <bubu at mikan.io> | Tue Nov 27 12:04:15 2018 +0100| [92b6657b0855981cc74fd652c0eac9e9d66f9987] | committer: Steve Lhomme

demux: mkv: Remove unnecessary throw

This case wasn't an error in the file itself and the playback can continue
 to the following files.

Additionally, throwing led to a crash on iOS 12.0.0 and iOS 12.0.1.

(ref: https://rink.hockeyapp.net/manage/apps/194470/app_versions/113/crash_reasons/242850559)

Signed-off-by: Steve Lhomme <robux4 at ycbcr.xyz>

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

 modules/demux/mkv/matroska_segment.cpp | 1 -
 1 file changed, 1 deletion(-)

diff --git a/modules/demux/mkv/matroska_segment.cpp b/modules/demux/mkv/matroska_segment.cpp
index 5a54dda79f..7706991d8d 100644
--- a/modules/demux/mkv/matroska_segment.cpp
+++ b/modules/demux/mkv/matroska_segment.cpp
@@ -1207,7 +1207,6 @@ int matroska_segment_c::BlockGet( KaxBlock * & pp_block, KaxSimpleBlock * & pp_s
         {
             VLC_UNUSED( kcue );
             msg_Warn( vars.p_demuxer, "find KaxCues FIXME" );
-            throw VLC_EGENERIC;
         }
         E_CASE_DEFAULT(element)
         {



More information about the vlc-commits mailing list