Address follow-up comments on #24

(cherry picked from commit fdf9b46)

Author: joelverhagen

src/NuGet.Server.Core/Infrastructure/ServerPackageRepository.cs

@@ -34,7 +34,7 @@ public class ServerPackageRepository
 
         private readonly bool _runBackgroundTasks;
         private FileSystemWatcher _fileSystemWatcher;
-        private bool _isFileSystemWatcherSuppressed = false;
+        private bool _isFileSystemWatcherSuppressed;
         private bool _needsRebuild = true;
 
         private Timer _persistenceTimer;
@@ -369,13 +369,23 @@ protected virtual void Dispose(bool disposing)
         }
 
         /// <summary>
-        /// Internal package cache containing packages metadata. 
+        /// Package cache containing packages metadata. 
         /// This data is generated if it does not exist already.
         /// </summary>
         private IEnumerable<ServerPackage> CachedPackages
         {
             get
             {
+                /*
+                 * We rebuild the package storage under either of two conditions:
+                 *
+                 * 1. If the "needs rebuild" flag is set to true. This is initially the case when the repository is
+                 *    instantiated, if a non-package drop file system event occurred (e.g. a file deletion), or if the
+                 *    cache was manually cleared.
+                 *
+                 * 2. If the store has no packages at all. This is so we pick up initial packages as quickly as
+                 *    possible.
+                 */
                 if (_needsRebuild || !_serverPackageStore.HasPackages())
                 {
                     lock (_syncLock)