minor: change some comment text in flotsam asset cache

bulletsim
Justin Clark-Casey (justincc) 2011-08-16 21:31:08 +01:00
parent 8c95c83562
commit fd3a7ab70c
1 changed files with 3 additions and 4 deletions

View File

@ -625,11 +625,10 @@ namespace Flotsam.RegionModules.AssetCache
// File.Copy(tempname, filename, true); // File.Copy(tempname, filename, true);
// File.Delete(tempname); // File.Delete(tempname);
// //
// For a brief period, this was done as a separate copy and then temporary file delete operation. // For a brief period, this was done as a separate copy and then temporary file delete operation to
// avoid an IOException caused by move if some competing thread had already written the file.
// However, this causes exceptions on Windows when other threads attempt to read a file // However, this causes exceptions on Windows when other threads attempt to read a file
// which is still being copied. So instead, go back to moving the file and swallowing any IOException // which is still being copied. So instead, go back to moving the file and swallow any IOException.
// that occurs because two threads race to cache the same data (and the second fails because the file
// already exists).
// //
// This situation occurs fairly rarely anyway. We assume in this that moves are atomic on the // This situation occurs fairly rarely anyway. We assume in this that moves are atomic on the
// filesystem. // filesystem.