Ver código fonte

Apply suggestions from code review

Co-authored-by: Claus Vium <cvium@users.noreply.github.com>
Nyanmisaka 4 anos atrás
pai
commit
060f6c194f

+ 1 - 1
MediaBrowser.MediaEncoding/Encoder/MediaEncoder.cs

@@ -605,9 +605,9 @@ namespace MediaBrowser.MediaEncoding.Encoder
             // Use ffmpeg to sample 100 (we can drop this if required using thumbnail=50 for 50 frames) frames and pick the best thumbnail. Have a fall back just in case.
             // Use ffmpeg to sample 100 (we can drop this if required using thumbnail=50 for 50 frames) frames and pick the best thumbnail. Have a fall back just in case.
             // mpegts need larger batch size otherwise the corrupted thumbnail will be created. Larger batch size will lower the processing speed.
             // mpegts need larger batch size otherwise the corrupted thumbnail will be created. Larger batch size will lower the processing speed.
             var enableThumbnail = useIFrame && !string.Equals("wtv", container, StringComparison.OrdinalIgnoreCase);
             var enableThumbnail = useIFrame && !string.Equals("wtv", container, StringComparison.OrdinalIgnoreCase);
-            var useLargerBatchSize = string.Equals("mpegts", container, StringComparison.OrdinalIgnoreCase);
             if (enableThumbnail)
             if (enableThumbnail)
             {
             {
+                var useLargerBatchSize = string.Equals("mpegts", container, StringComparison.OrdinalIgnoreCase);
                 var batchSize = useLargerBatchSize ? "50" : "24";
                 var batchSize = useLargerBatchSize ? "50" : "24";
                 if (string.IsNullOrEmpty(vf))
                 if (string.IsNullOrEmpty(vf))
                 {
                 {