Parcourir la source

Apply review suggestions

Shadowghost il y a 2 ans
Parent
commit
f2b7f664aa

+ 1 - 1
Jellyfin.Api/Helpers/TranscodingJobHelper.cs

@@ -325,7 +325,7 @@ public class TranscodingJobHelper : IDisposable
             await DeletePartialStreamFiles(job.Path!, job.Type, 0, 1500).ConfigureAwait(false);
             if (job.MediaSource?.VideoType == VideoType.Dvd || job.MediaSource?.VideoType == VideoType.BluRay)
             {
-                var path = Path.GetDirectoryName(job.Path) + "/" + job.MediaSource.Id + ".concat";
+                var path = Path.Join(job.Path, "/" + job.MediaSource.Id + ".concat");
                 File.Delete(path);
             }
         }

+ 5 - 3
MediaBrowser.Controller/MediaEncoding/EncodingHelper.cs

@@ -943,10 +943,12 @@ namespace MediaBrowser.Controller.MediaEncoding
 
             if (state.MediaSource.VideoType == VideoType.Dvd || state.MediaSource.VideoType == VideoType.BluRay)
             {
-                var tmpConcatPath = options.TranscodingTempPath + "/" + state.MediaSource.Id + ".concat";
+                var tmpConcatPath = Path.Join(options.TranscodingTempPath, "/" + state.MediaSource.Id + ".concat");
                 _mediaEncoder.GenerateConcatConfig(state.MediaSource, tmpConcatPath);
-                arg.Append(" -f concat -safe 0 ");
-                arg.Append(" -i " + tmpConcatPath + " ");
+                arg.Append(" -f concat -safe 0 ")
+                    .Append(" -i ")
+                    .Append(tmpConcatPath)
+                    .Append(' ');
             }
             else
             {

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

@@ -450,7 +450,7 @@ namespace MediaBrowser.MediaEncoding.Encoder
                 prefix = "bluray";
             }
 
-            return EncodingUtils.GetInputArgument(prefix, new List<string>() { inputFile }, mediaSource.Protocol);
+            return EncodingUtils.GetInputArgument(prefix, new[] { inputFile }, mediaSource.Protocol);
         }
 
         /// <inheritdoc />
@@ -458,7 +458,7 @@ namespace MediaBrowser.MediaEncoding.Encoder
         {
             const string Prefix = "file";
 
-            return EncodingUtils.GetInputArgument(Prefix, new List<string> { inputFile }, MediaProtocol.File);
+            return EncodingUtils.GetInputArgument(Prefix, new[] { inputFile }, MediaProtocol.File);
         }
 
         /// <summary>