Wrap expressions for improved readability

Author: sharwell

StyleCop.Analyzers/StyleCop.Analyzers.CodeFixes/DocumentationRules/SA1617CodeFixProvider.cs

@@ -83,7 +83,8 @@ private static async Task<Document> GetTransformedDocumentAsync(Document documen
             List<SyntaxNode> nodesToFix = new List<SyntaxNode>();
             nodesToFix.Add(returnsElement);
 
-            if (previous is XmlTextSyntax previousAsTextSyntax && XmlCommentHelper.IsConsideredEmpty(previousAsTextSyntax))
+            if (previous is XmlTextSyntax previousAsTextSyntax
+                && XmlCommentHelper.IsConsideredEmpty(previousAsTextSyntax))
             {
                 nodesToFix.Add(previous);
             }

StyleCop.Analyzers/StyleCop.Analyzers/DocumentationRules/SA1615ElementReturnValueMustBeDocumented.cs

@@ -84,7 +84,8 @@ private static void HandleDeclaration(SyntaxNodeAnalysisContext context, bool ne
                 return;
             }
 
-            if (returnType is PredefinedTypeSyntax predefinedType && predefinedType.Keyword.IsKind(SyntaxKind.VoidKeyword))
+            if (returnType is PredefinedTypeSyntax predefinedType
+                && predefinedType.Keyword.IsKind(SyntaxKind.VoidKeyword))
             {
                 // There is no return value
                 return;

StyleCop.Analyzers/StyleCop.Analyzers/DocumentationRules/StandardTextDiagnosticBase.cs

@@ -95,7 +95,9 @@ protected static MatchResult HandleDeclaration(SyntaxNodeAnalysisContext context
                 var summaryNodes = summaryElement.Nodes().ToList();
                 if (summaryNodes.Count >= 3)
                 {
-                    if (summaryNodes[0] is XText firstTextPartNode && summaryNodes[1] is XElement classReferencePart && summaryNodes[2] is XText secondTextPartNode)
+                    if (summaryNodes[0] is XText firstTextPartNode
+                        && summaryNodes[1] is XElement classReferencePart
+                        && summaryNodes[2] is XText secondTextPartNode)
                     {
                         if (TextPartsMatch(firstTextPart, secondTextPart, firstTextPartNode, secondTextPartNode))
                         {
@@ -122,7 +124,9 @@ protected static MatchResult HandleDeclaration(SyntaxNodeAnalysisContext context
                 if (summaryElement.Content.Count >= 3)
                 {
                     // Standard text has the form <part1><see><part2>
-                    if (summaryElement.Content[0] is XmlTextSyntax firstTextPartSyntax && summaryElement.Content[1] is XmlEmptyElementSyntax classReferencePart && summaryElement.Content[2] is XmlTextSyntax secondTextPartSyntax)
+                    if (summaryElement.Content[0] is XmlTextSyntax firstTextPartSyntax
+                        && summaryElement.Content[1] is XmlEmptyElementSyntax classReferencePart
+                        && summaryElement.Content[2] is XmlTextSyntax secondTextPartSyntax)
                     {
                         if (TextPartsMatch(firstTextPart, secondTextPart, firstTextPartSyntax, secondTextPartSyntax))
                         {

StyleCop.Analyzers/StyleCop.Analyzers/Helpers/UsingDirectiveSyntaxHelpers.cs

@@ -104,7 +104,8 @@ internal static bool StartsWithAlias(this UsingDirectiveSyntax usingDirective, S
         {
             foreach (var identifier in usingDirective.DescendantNodes())
             {
-                if (identifier is IdentifierNameSyntax identifierName && ExcludeGlobalKeyword(identifierName))
+                if (identifier is IdentifierNameSyntax identifierName
+                    && ExcludeGlobalKeyword(identifierName))
                 {
                     return identifierName.Identifier;
                 }

StyleCop.Analyzers/StyleCop.Analyzers/LayoutRules/SA1518UseLineEndingsCorrectlyAtEndOfFile.cs

@@ -121,7 +121,8 @@ private static void HandleSyntaxTree(SyntaxTreeAnalysisContext context, StyleCop
 
             if (precedingTrivia.IsDirective)
             {
-                if (precedingTrivia.GetStructure() is DirectiveTriviaSyntax directiveTriviaSyntax && directiveTriviaSyntax.EndOfDirectiveToken.HasTrailingTrivia)
+                if (precedingTrivia.GetStructure() is DirectiveTriviaSyntax directiveTriviaSyntax
+                    && directiveTriviaSyntax.EndOfDirectiveToken.HasTrailingTrivia)
                 {
                     var trailingWhitespaceIndex = TriviaHelper.IndexOfTrailingWhitespace(directiveTriviaSyntax.EndOfDirectiveToken.TrailingTrivia);
                     if (trailingWhitespaceIndex >= 0)

StyleCop.Analyzers/StyleCop.Analyzers/MaintainabilityRules/SA1119StatementMustNotUseUnnecessaryParenthesis.cs

@@ -153,7 +153,8 @@ private static void HandleParenthesizedExpression(SyntaxNodeAnalysisContext cont
                     }
                     else
                     {
-                        if (node.Parent is EqualsValueClauseSyntax equalsValue && equalsValue.Value == node)
+                        if (node.Parent is EqualsValueClauseSyntax equalsValue
+                            && equalsValue.Value == node)
                         {
                             ReportDiagnostic(context, node);
                         }

StyleCop.Analyzers/StyleCop.Analyzers/ReadabilityRules/SA1101PrefixLocalCallsWithThis.cs

@@ -177,7 +177,8 @@ private static void HandleIdentifierNameImpl(SyntaxNodeAnalysisContext context,
                     return;
                 }
 
-                if (symbol is IMethodSymbol methodSymbol && methodSymbol.MethodKind == MethodKind.Constructor)
+                if (symbol is IMethodSymbol methodSymbol
+                    && methodSymbol.MethodKind == MethodKind.Constructor)
                 {
                     return;
                 }

StyleCop.Analyzers/StyleCop.Analyzers/ReadabilityRules/SA1122UseStringEmptyForEmptyStrings.cs

@@ -100,12 +100,14 @@ private static bool HasToBeConstant(LiteralExpressionSyntax literalExpression)
                     return false;
                 }
 
-                if (variableDeclarationSyntax.Parent is FieldDeclarationSyntax fieldDeclarationSyntax && fieldDeclarationSyntax.Modifiers.Any(SyntaxKind.ConstKeyword))
+                if (variableDeclarationSyntax.Parent is FieldDeclarationSyntax fieldDeclarationSyntax
+                    && fieldDeclarationSyntax.Modifiers.Any(SyntaxKind.ConstKeyword))
                 {
                     return true;
                 }
 
-                if (variableDeclarationSyntax.Parent is LocalDeclarationStatementSyntax localDeclarationStatementSyntax && localDeclarationStatementSyntax.Modifiers.Any(SyntaxKind.ConstKeyword))
+                if (variableDeclarationSyntax.Parent is LocalDeclarationStatementSyntax localDeclarationStatementSyntax
+                    && localDeclarationStatementSyntax.Modifiers.Any(SyntaxKind.ConstKeyword))
                 {
                     return true;
                 }