Add comments

zhengbli · zhengbli · commit b155fa847a6d · 2016-03-11T16:18:40.000-08:00
diff --git a/src/compiler/sys.ts b/src/compiler/sys.ts
@@ -500,6 +500,9 @@ namespace ts {
                     for (const current of files) {
                         const name = combinePaths(path, current);
                         if (!contains(exclude, getCanonicalPath(name))) {
+                            // fs.statSync would throw an exception if the file is a symlink
+                            // whose linked file doesn't exist. fs.lstatSync would return a stat
+                            // object for the symlink file itself in this case
                             const stat = _fs.lstatSync(name);
                             if (stat.isFile()) {
                                 if (!extension || fileExtensionIs(name, extension)) {
diff --git a/src/server/editorServices.ts b/src/server/editorServices.ts
@@ -1273,15 +1273,14 @@ namespace ts.server {
                     return error;
                 }
                 else {
+                    // if the project is too large, the root files might not have been all loaded if the total
+                    // program size reached the upper limit. In that case project.projectOptions.files should 
+                    // be more precise. However this would only happen for configured project.
                     const oldFileNames = project.projectOptions ? project.projectOptions.files : project.compilerService.host.roots.map(info => info.fileName);
                     const newFileNames = projectOptions.files;
                     const fileNamesToRemove = oldFileNames.filter(f => newFileNames.indexOf(f) < 0);
                     const fileNamesToAdd = newFileNames.filter(f => oldFileNames.indexOf(f) < 0);
 
-                    if (fileNamesToAdd.length === 0 && fileNamesToRemove.length === 0) {
-                        return;
-                    }
-
                     for (const fileName of fileNamesToRemove) {
                         const info = this.getScriptInfo(fileName);
                         if (info) {
