[vlc-devel] [PATCH] mkv: removed duplicate case for S_VOBSUB

Filip Roséen filip at videolabs.io
Mon Mar 14 09:11:47 CET 2016


During the refactor to use the new StringDispatcher it seems the code
responsible for detecting the S_VOBSUB codec was added twice (but logic
only for the first entry).

Even though the first entry will take precedence (and hence make it
work), we should of course not have unused and unnecessary code in the
code base.

This patch removes the duplicate entry.
---
 modules/demux/mkv/matroska_segment_parse.cpp | 3 ---
 1 file changed, 3 deletions(-)

diff --git a/modules/demux/mkv/matroska_segment_parse.cpp b/modules/demux/mkv/matroska_segment_parse.cpp
index 16e93d6..41bcc78 100644
--- a/modules/demux/mkv/matroska_segment_parse.cpp
+++ b/modules/demux/mkv/matroska_segment_parse.cpp
@@ -1724,9 +1724,6 @@ int32_t matroska_segment_c::TrackInit( mkv_track_t * p_tk )
                 }
             }
         }
-        S_CASE("S_VOBSUB") {
-            VLC_UNUSED(vars);
-        }
         S_CASE("S_HDMV/PGS") {
             vars.p_fmt->i_codec = VLC_CODEC_BD_PG;
         }
-- 
2.7.2



More information about the vlc-devel mailing list