Fix NRE in UseSupportsShouldProcess rule (#762)

Kapil Borle · web-flow · commit 27e05c39478f · 2017-05-17T20:22:37.000-07:00
diff --git a/rules/UseSupportsShouldProcess.cs b/rules/UseSupportsShouldProcess.cs
@@ -65,7 +65,7 @@ private List<DiagnosticRecord> FindViolations()
                 int whatIfIndex, confirmIndex;
                 ParameterAst whatIfParamAst, confirmParamAst;
                 ParamBlockAst paramBlockAst;
-                var parameterAsts = functionDefinitionAst.GetParameterAsts(out paramBlockAst).ToArray();
+                var parameterAsts = functionDefinitionAst.GetParameterAsts(out paramBlockAst)?.ToArray();
                 if (parameterAsts == null)
                 {
                     continue;

Original file line number	Diff line number	Diff line change
`@@ -65,7 +65,7 @@ private List<DiagnosticRecord> FindViolations()`
`65`	`65`	`int whatIfIndex, confirmIndex;`
`66`	`66`	`ParameterAst whatIfParamAst, confirmParamAst;`
`67`	`67`	`ParamBlockAst paramBlockAst;`
`68`		`- var parameterAsts = functionDefinitionAst.GetParameterAsts(out paramBlockAst).ToArray();`
	`68`	`+ var parameterAsts = functionDefinitionAst.GetParameterAsts(out paramBlockAst)?.ToArray();`
`69`	`69`	`if (parameterAsts == null)`
`70`	`70`	`{`
`71`	`71`	`continue;`