Don't flow through to rest of codepath with Ast and ProjectEntry are null

Author: mousetraps

Nodejs/Product/Analysis/Analysis/Analyzer/AnalysisUnit.cs

@@ -180,7 +180,7 @@ internal void Analyze(DDG ddg, CancellationToken cancel) {
                 long endTime = _sw.ElapsedMilliseconds;
                 var thisTime = endTime - startTime;
                 _analysisTime += thisTime;
-                if (thisTime >= 500 || (_analysisTime / _analysisCount) > 500) {                
+                if (thisTime >= 500 || (_analysisTime / _analysisCount) > 500) {
                     Trace.TraceWarning("Analyzed: {0} {1} ({2} count, {3}ms total, {4}ms mean)", this, thisTime, _analysisCount, _analysisTime, (double)_analysisTime / _analysisCount);
                 }
             }
@@ -191,7 +191,7 @@ internal virtual void AnalyzeWorker(DDG ddg, CancellationToken cancel) {
             Debug.Assert(Ast != null, "Ast has unexpected null value");
             Debug.Assert(ProjectEntry != null, "ProjectEntry has unexpected null value");
 
-            if (Ast != null && ProjectEntry != null && Tree != ProjectEntry.Tree) {
+            if (Ast == null || ProjectEntry == null || Tree != ProjectEntry.Tree) {
                 // analysis unit properties are invalid or we were enqueued and a new version became available
                 // don't re-analyze against the old version.
                 return;