reduce memory usage from file system info

pull/702/head
Luke Pulverenti 12 years ago
parent 39e3092c52
commit 830e5c01f1

@ -37,14 +37,16 @@ namespace MediaBrowser.Controller.IO
{ {
var isDirectory = (entry.Attributes & FileAttributes.Directory) == FileAttributes.Directory; var isDirectory = (entry.Attributes & FileAttributes.Directory) == FileAttributes.Directory;
if (resolveShortcuts && FileSystem.IsShortcut(entry.FullName)) var fullName = entry.FullName;
if (resolveShortcuts && FileSystem.IsShortcut(fullName))
{ {
var newPath = FileSystem.ResolveShortcut(entry.FullName); var newPath = FileSystem.ResolveShortcut(fullName);
if (string.IsNullOrWhiteSpace(newPath)) if (string.IsNullOrWhiteSpace(newPath))
{ {
//invalid shortcut - could be old or target could just be unavailable //invalid shortcut - could be old or target could just be unavailable
logger.Warn("Encountered invalid shortcut: " + entry.FullName); logger.Warn("Encountered invalid shortcut: " + fullName);
continue; continue;
} }
@ -57,18 +59,18 @@ namespace MediaBrowser.Controller.IO
args.AddAdditionalLocation(newPath); args.AddAdditionalLocation(newPath);
} }
dict[data.FullName] = data; dict[newPath] = data;
} }
else if (flattenFolderDepth > 0 && isDirectory) else if (flattenFolderDepth > 0 && isDirectory)
{ {
foreach (var child in GetFilteredFileSystemEntries(entry.FullName, logger, flattenFolderDepth: flattenFolderDepth - 1, resolveShortcuts: resolveShortcuts)) foreach (var child in GetFilteredFileSystemEntries(fullName, logger, flattenFolderDepth: flattenFolderDepth - 1, resolveShortcuts: resolveShortcuts))
{ {
dict[child.Key] = child.Value; dict[child.Key] = child.Value;
} }
} }
else else
{ {
dict[entry.FullName] = entry; dict[fullName] = entry;
} }
} }

@ -112,7 +112,7 @@ namespace MediaBrowser.Controller.Library
return false; return false;
} }
var parentDir = System.IO.Path.GetDirectoryName(FileInfo.FullName) ?? string.Empty; var parentDir = System.IO.Path.GetDirectoryName(Path) ?? string.Empty;
return (parentDir.Length > _appPaths.RootFolderPath.Length return (parentDir.Length > _appPaths.RootFolderPath.Length
&& parentDir.StartsWith(_appPaths.RootFolderPath, StringComparison.OrdinalIgnoreCase)); && parentDir.StartsWith(_appPaths.RootFolderPath, StringComparison.OrdinalIgnoreCase));

@ -179,7 +179,7 @@ namespace MediaBrowser.Controller.Library
private static bool IsSeasonFolder(string path) private static bool IsSeasonFolder(string path)
{ {
// It's a season folder if it's named as such and does not contain any audio files, apart from theme.mp3 // It's a season folder if it's named as such and does not contain any audio files, apart from theme.mp3
return GetSeasonNumberFromPath(path) != null && !new DirectoryInfo(path).EnumerateFiles().Any(i => EntityResolutionHelper.IsAudioFile(i.FullName) && !string.Equals(Path.GetFileNameWithoutExtension(i.Name), BaseItem.ThemeSongFilename)); return GetSeasonNumberFromPath(path) != null && !Directory.EnumerateFiles(path).Any(i => EntityResolutionHelper.IsAudioFile(i) && !string.Equals(Path.GetFileNameWithoutExtension(i), BaseItem.ThemeSongFilename));
} }
/// <summary> /// <summary>
@ -223,7 +223,9 @@ namespace MediaBrowser.Controller.Library
} }
else else
{ {
if (EntityResolutionHelper.IsVideoFile(child.FullName) && GetEpisodeNumberFromFile(child.FullName, false).HasValue) var fullName = child.FullName;
if (EntityResolutionHelper.IsVideoFile(fullName) && GetEpisodeNumberFromFile(fullName, false).HasValue)
{ {
return true; return true;
} }
@ -275,7 +277,7 @@ namespace MediaBrowser.Controller.Library
} }
private static readonly CultureInfo UsCulture = new CultureInfo("en-US"); private static readonly CultureInfo UsCulture = new CultureInfo("en-US");
private static int? ParseEpisodeNumber(string val) private static int? ParseEpisodeNumber(string val)
{ {
int num; int num;

Loading…
Cancel
Save