Merge pull request #3576 from HelloWorld017/fix/sami-utf16

Fix SAMI UTF-16 Encoding Bug
5 years ago · 0cb2cd9456
parent 83509e99cb 6e069f925b
commit 0cb2cd9456
1 changed files with 3 additions and 1 deletions
--- a/MediaBrowser.MediaEncoding/Subtitles/SubtitleEncoder.cs
+++ b/MediaBrowser.MediaEncoding/Subtitles/SubtitleEncoder.cs
@ -426,7 +426,9 @@ namespace MediaBrowser.MediaEncoding.Subtitles

            // FFmpeg automatically convert character encoding when it is UTF-16
            // If we specify character encoding, it rejects with "do not specify a character encoding" and "Unable to recode subtitle event"
-            if ((inputPath.EndsWith(".smi") || inputPath.EndsWith(".sami")) && (encodingParam == "UTF-16BE" || encodingParam == "UTF-16LE"))
+            if ((inputPath.EndsWith(".smi") || inputPath.EndsWith(".sami")) &&
+                (encodingParam.Equals("UTF-16BE", StringComparison.OrdinalIgnoreCase) ||
+                 encodingParam.Equals("UTF-16LE", StringComparison.OrdinalIgnoreCase)))
            {
                encodingParam = "";
            }