Merge pull request #3038 from LLLXXXCCC/RemoveExtraValueContentAnalysis

Remove extra value content analysis
dotnet · Nov 13, 2019 · 1f5001e · 1f5001e
2 parents dbc50b1 + 98d43b0
commit 1f5001e
Showing 1 changed file with 2 additions and 1 deletion.
diff --git a/src/Utilities/FlowAnalysis/FlowAnalysis/Analysis/TaintedDataAnalysis/SourceInfo.cs b/src/Utilities/FlowAnalysis/FlowAnalysis/Analysis/TaintedDataAnalysis/SourceInfo.cs
@@ -2,6 +2,7 @@
 
 using System;
 using System.Collections.Immutable;
+using System.Linq;
 using Microsoft.CodeAnalysis.FlowAnalysis.DataFlow.PointsToAnalysis;
 using Microsoft.CodeAnalysis.FlowAnalysis.DataFlow.ValueContentAnalysis;
 using Microsoft.CodeAnalysis.Operations;
@@ -140,7 +141,7 @@ internal class SourceInfo : ITaintedDataInfo, IEquatable<SourceInfo>
         /// <summary>
         /// Indicates that this <see cref="SourceInfo"/> uses <see cref="ValueContentAbstractValue"/>s.
         /// </summary>
-        public bool RequiresValueContentAnalysis => this.TaintedMethodsNeedsValueContentAnalysis != null;
+        public bool RequiresValueContentAnalysis => this.TaintedMethodsNeedsValueContentAnalysis.Any();
 
         public override int GetHashCode()
         {