ソースを参照

better disposal around using

Luke Pulverenti 12 年 前
コミット
196f3eb574

+ 9 - 9
MediaBrowser.Server.Implementations/Providers/ProviderManager.cs

@@ -468,17 +468,17 @@ namespace MediaBrowser.Server.Implementations.Providers
 
             try
             {
-                // If the file already exists but is hidden, the below save will throw an UnauthorizedAccessException
-                var existingFileInfo = new FileInfo(path);
-
-                if (existingFileInfo.Exists && existingFileInfo.Attributes.HasFlag(FileAttributes.Hidden))
+                using (dataToSave)
                 {
-                    existingFileInfo.Delete();
-                }
+                    // If the file already exists but is hidden, the below save will throw an UnauthorizedAccessException
+                    var existingFileInfo = new FileInfo(path);
 
-                using (var fs = new FileStream(path, FileMode.Create, FileAccess.Write, FileShare.Read, StreamDefaults.DefaultFileStreamBufferSize, FileOptions.Asynchronous))
-                {
-                    using (dataToSave)
+                    if (existingFileInfo.Exists && existingFileInfo.Attributes.HasFlag(FileAttributes.Hidden))
+                    {
+                        existingFileInfo.Delete();
+                    }
+
+                    using (var fs = new FileStream(path, FileMode.Create, FileAccess.Write, FileShare.Read, StreamDefaults.DefaultFileStreamBufferSize, FileOptions.Asynchronous))
                     {
                         await dataToSave.CopyToAsync(fs, StreamDefaults.DefaultCopyToBufferSize, cancellationToken).ConfigureAwait(false);
                     }