Minor changes to methods that implement finding wildcard or null in an ast in UseToExportFieldsInManifest.

Author: Kapil Borle

Rules/UseToExportFieldsInManifest.cs

@@ -93,14 +93,7 @@ private bool IsValidManifest(Ast ast, string fileName)
         private bool HasWildcardInExpression(Ast ast)
         {
             var strConstExprAst = ast as StringConstantExpressionAst;
-            if (strConstExprAst != null && WildcardPattern.ContainsWildcardCharacters(strConstExprAst.Value))
-            {
-                return true;
-            }
-            else
-            {
-                return false;
-            }
+            return strConstExprAst != null && WildcardPattern.ContainsWildcardCharacters(strConstExprAst.Value);
         }
 
         /// <summary>
@@ -111,16 +104,9 @@ private bool HasWildcardInExpression(Ast ast)
         private bool HasNullInExpression(Ast ast)
         {
             var varExprAst = ast as VariableExpressionAst;
-            if (varExprAst != null
-                && varExprAst.VariablePath.IsUnqualified
-                && varExprAst.VariablePath.UserPath.Equals("null", StringComparison.OrdinalIgnoreCase))
-            {
-                return true;
-            }
-            else
-            {
-                return false;
-            }
+            return varExprAst != null
+                    && varExprAst.VariablePath.IsUnqualified
+                    && varExprAst.VariablePath.UserPath.Equals("null", StringComparison.OrdinalIgnoreCase);
         }
 
         /// <summary>