Merge pull request #525 from mousetraps/i467

fix #467 VS hangs after closing NTVS project

Author: mousetraps

Nodejs/Product/Analysis/Analysis/AnalysisLimits.cs

@@ -34,6 +34,7 @@ public AnalysisLimits() {
             MaxObjectLiteralProperties = 50;
             MaxObjectKeysTypes = 5;
             MaxMergeTypes = 5;
+            MaxEvents = 5;
 
             // There no practical reasons to go deeper in dependencies analysis.
             // Number 4 is very practical. Here the examples which hightlight idea.
@@ -147,7 +148,12 @@ public static AnalysisLimits MakeLowAnalysisLimits() {
         /// <summary>
         /// Gets the maximum number of types which can be merged at once.
         /// </summary>
-        public int MaxMergeTypes { get; set; }
+        public int MaxMergeTypes { get; set; }
+
+        /// <summary>
+        /// Gets the maximum number of events which can be emitted at once.
+        /// </summary>
+        public int MaxEvents { get; set; }
 
         /// <summary>
         /// Gets the maximum level of dependency modules which could be analyzed.

Nodejs/Product/Analysis/Analysis/GlobalBuilder.cs

@@ -1310,7 +1310,7 @@ private static IAnalysisSet DefineProperty(FunctionValue func, Node node, Analys
 
         private static IAnalysisSet DefineProperties(FunctionValue func, Node node, AnalysisUnit unit, IAnalysisSet @this, IAnalysisSet[] args) {
             // object, {propName: {desc}, ...}
-            if (args.Length >= 2) {
+            if (args.Length >= 2 && args[0].Count < unit.Analyzer.Limits.MaxMergeTypes) {
                 foreach (var obj in args[0]) {
                     ExpandoValue target = obj.Value as ExpandoValue;
                     if (target != null) {
@@ -1342,8 +1342,14 @@ private static IAnalysisSet DefineProperties(FunctionValue func, Node node, Anal
         }
 
         private static IAnalysisSet Require(FunctionValue func, Node node, AnalysisUnit unit, IAnalysisSet @this, IAnalysisSet[] args) {
-            CallNode call = (CallNode)node;
-            IAnalysisSet res = AnalysisSet.Empty;
+            IAnalysisSet res = AnalysisSet.Empty;
+
+            if (node.GetType() != typeof(CallNode)) {
+                return res;
+            }
+
+            var call = (CallNode)node;
+
             // we care a lot about require analysis and people do some pretty
             // crazy dynamic things for require calls.  If we let our normal
             // analysis and specialized function handle it we won't get things

Nodejs/Product/Analysis/Analysis/NodejsModuleBuilder.Specializations.cs

@@ -451,7 +451,7 @@ private static IAnalysisSet EventEmitterEmit(FunctionValue func, Node node, Anal
                 if (@this != null) {
                     foreach (var thisArg in @this) {
                         ExpandoValue expando = @thisArg.Value as ExpandoValue;
-                        if (expando != null) {
+                        if (expando != null && args[0].Count < unit.Analyzer.Limits.MaxEvents) {
                             foreach (var arg in args[0]) {
                                 var strValue = arg.Value.GetStringValue();
                                 if (strValue != null) {