Expand things a bit to call out that there is a 'this' parameter bein…

…g read
dotnet · Aug 1, 2019 · 1770001 · 1770001
1 parent 140b0eb
commit 1770001
Showing 1 changed file with 2 additions and 1 deletion.
diff --git a/src/Features/Core/Portable/ExtractMethod/MethodExtractor.Analyzer.cs b/src/Features/Core/Portable/ExtractMethod/MethodExtractor.Analyzer.cs
@@ -91,7 +91,8 @@ public async Task<AnalyzerResult> AnalyzeAsync()
 
                 var instanceMemberIsUsed = thisParameterBeingRead != null || isThisParameterWritten;
                 var shouldBeReadOnly = !isThisParameterWritten
-                    && thisParameterBeingRead?.Type is { TypeKind: TypeKind.Struct, IsReadOnly: false };
+                    && thisParameterBeingRead != null
+                    && thisParameterBeingRead.Type is { TypeKind: TypeKind.Struct, IsReadOnly: false };
 
                 // check whether end of selection is reachable
                 var endOfSelectionReachable = IsEndOfSelectionReachable(model);