Fix filelist compare bug

Author: veblush

core/IncrementalCompiler/Compiler.cs

@@ -39,6 +39,7 @@ private CompileResult BuildFull(CompileOptions options)
         {
             var result = new CompileResult();
 
+            _logger.Info("BuildFull");
             _options = options;
 
             _referenceFileList = new FileTimeList();
@@ -71,28 +72,30 @@ private CompileResult BuildIncremental(CompileOptions options)
         {
             var result = new CompileResult();
 
+            _logger.Info("BuildIncremental");
             _options = options;
 
-            // TODO: guard failure of compilation, ...
-
             // update reference files
 
             var referenceChanges = _referenceFileList.Update(options.References);
             foreach (var file in referenceChanges.Added)
             {
+                _logger.Info("+ {0}", file);
                 var reference = CreateReference(file);
                 _compilation = _compilation.AddReferences(reference);
                 _referenceMap.Add(file, reference);
             }
             foreach (var file in referenceChanges.Changed)
             {
+                _logger.Info("* {0}", file);
                 var reference = CreateReference(file);
                 _compilation = _compilation.RemoveReferences(_referenceMap[file])
                                            .AddReferences(reference);
                 _referenceMap[file] = reference;
             }
             foreach (var file in referenceChanges.Removed)
             {
+                _logger.Info("- {0}", file);
                 _compilation = _compilation.RemoveReferences(_referenceMap[file]);
                 _referenceMap.Remove(file);
             }
@@ -103,19 +106,22 @@ private CompileResult BuildIncremental(CompileOptions options)
             var parseOption = new CSharpParseOptions(LanguageVersion.CSharp6, DocumentationMode.Parse, SourceCodeKind.Regular, options.Defines);
             foreach (var file in sourceChanges.Added)
             {
+                _logger.Info("+ {0}", file);
                 var syntaxTree = ParseSource(file, parseOption);
                 _compilation = _compilation.AddSyntaxTrees(syntaxTree);
                 _sourceMap.Add(file, syntaxTree);
             }
             foreach (var file in sourceChanges.Changed)
             {
+                _logger.Info("* {0}", file);
                 var syntaxTree = ParseSource(file, parseOption);
                 _compilation = _compilation.RemoveSyntaxTrees(_sourceMap[file])
                                            .AddSyntaxTrees(syntaxTree);
                 _sourceMap[file] = syntaxTree;
             }
             foreach (var file in sourceChanges.Removed)
             {
+                _logger.Info("- {0}", file);
                 _compilation = _compilation.RemoveSyntaxTrees(_sourceMap[file]);
                 _sourceMap.Remove(file);
             }

core/IncrementalCompiler/CompilerService.cs

@@ -25,10 +25,10 @@ public CompileResult Build(string projectPath, CompileOptions options)
 
             if (string.IsNullOrEmpty(_projectPath) || _projectPath != projectPath)
             {
-                // Flush existing
-
+                // create new one
                 _compilerMap = new Dictionary<string, Compiler>();
-                _logger.Info("Flush old project. (Project={0})", _projectPath);
+                if (string.IsNullOrEmpty(_projectPath) == false)
+                    _logger.Info("Flush old project. (Project={0})", _projectPath);
             }
 
             _projectPath = projectPath;

core/IncrementalCompiler/FileTimeList.cs

@@ -46,7 +46,7 @@ public Result Update(IEnumerable<Tuple<string, DateTime>> files)
             var j = 0;
             while (i < _files.Count && j < oldFiles.Count)
             {
-                var c = Comparer<string>.Default.Compare(_files[i].Item1, oldFiles[i].Item1);
+                var c = Comparer<string>.Default.Compare(_files[i].Item1, oldFiles[j].Item1);
                 if (c == 0)
                 {
                     if (_files[i].Item2 != oldFiles[j].Item2)