Changed ExpressionPrettyPrinter and ExpressionVisitor to put methods in alphabetical order.

Author: rstam

Driver/Linq/Expressions/ExpressionPrettyPrinter.cs

@@ -56,33 +56,6 @@ public string PrettyPrint(Expression node)
         }
 
         // protected methods
-        /// <summary>
-        /// Visits an ElementInit node.
-        /// </summary>
-        /// <param name="node">The ElementInit node.</param>
-        /// <returns>The ElementInit node.</returns>
-        protected override ElementInit VisitElementInit(ElementInit node)
-        {
-            throw new NotImplementedException();
-        }
-
-        /// <summary>
-        /// Visits a UnaryExpression.
-        /// </summary>
-        /// <param name="node">The UnaryExpression.</param>
-        /// <returns>The UnaryExpression.</returns>
-        protected override Expression VisitUnary(UnaryExpression node)
-        {
-            WriteHeader(node);
-            using (new Indentation(this))
-            {
-                WriteLine("Method={0}", node.Method == null ? "null" : node.Method.Name);
-                WriteLine("Operand:");
-                VisitIndented(node.Operand);
-            }
-            return node;
-        }
-
         /// <summary>
         /// Visits a BinaryExpression.
         /// </summary>
@@ -103,11 +76,11 @@ protected override Expression VisitBinary(BinaryExpression node)
         }
 
         /// <summary>
-        /// Visits a TypeBinaryExpression.
+        /// Visits a ConditionalExpression.
         /// </summary>
-        /// <param name="node">The TypeBinaryExpression.</param>
-        /// <returns>The TypeBinaryExpression.</returns>
-        protected override Expression VisitTypeBinary(TypeBinaryExpression node)
+        /// <param name="node">The ConditionalExpression.</param>
+        /// <returns>The ConditionalExpression.</returns>
+        protected override Expression VisitConditional(ConditionalExpression node)
         {
             throw new NotImplementedException();
         }
@@ -128,31 +101,67 @@ protected override Expression VisitConstant(ConstantExpression node)
         }
 
         /// <summary>
-        /// Visits a ConditionalExpression.
+        /// Visits an ElementInit node.
         /// </summary>
-        /// <param name="node">The ConditionalExpression.</param>
-        /// <returns>The ConditionalExpression.</returns>
-        protected override Expression VisitConditional(ConditionalExpression node)
+        /// <param name="node">The ElementInit node.</param>
+        /// <returns>The ElementInit node.</returns>
+        protected override ElementInit VisitElementInit(ElementInit node)
         {
             throw new NotImplementedException();
         }
 
         /// <summary>
-        /// Visits a ParameterExpression.
+        /// Visits an ElementInit list.
         /// </summary>
-        /// <param name="node">The ParameterExpression.</param>
-        /// <returns>The ParameterExpression.</returns>
-        protected override Expression VisitParameter(ParameterExpression node)
+        /// <param name="nodes">The ElementInit list.</param>
+        /// <returns>The ElementInit list.</returns>
+        protected override IEnumerable<ElementInit> VisitElementInitList(
+            ReadOnlyCollection<ElementInit> nodes)
+        {
+            throw new NotImplementedException();
+        }
+
+        /// <summary>
+        /// Visits an InvocationExpression.
+        /// </summary>
+        /// <param name="node">The InvocationExpression.</param>
+        /// <returns>The InvocationExpression.</returns>
+        protected override Expression VisitInvocation(InvocationExpression node)
+        {
+            throw new NotImplementedException();
+        }
+
+        /// <summary>
+        /// Visits a LambdaExpression.
+        /// </summary>
+        /// <param name="node">The LambdaExpression.</param>
+        /// <returns>The LambdaExpression.</returns>
+        protected override Expression VisitLambda(LambdaExpression node)
         {
             WriteHeader(node);
             using (new Indentation(this))
             {
-                WriteLine("Name={0}", node.Name);
-                WriteLine("Type={0}", FriendlyClassName(node.Type));
+                WriteLine("Parameters:");
+                foreach (var parameter in node.Parameters)
+                {
+                    VisitIndented(parameter);
+                }
+                WriteLine("Body:");
+                VisitIndented(node.Body);
             }
             return node;
         }
 
+        /// <summary>
+        /// Visits a ListInitExpression.
+        /// </summary>
+        /// <param name="node">The ListInitExpression.</param>
+        /// <returns>The ListInitExpression.</returns>
+        protected override Expression VisitListInit(ListInitExpression node)
+        {
+            throw new NotImplementedException();
+        }
+
         /// <summary>
         /// Visits a MemberExpression.
         /// </summary>
@@ -170,29 +179,6 @@ protected override Expression VisitMember(MemberExpression node)
             return node;
         }
 
-        /// <summary>
-        /// Visits a MethodCallExpression.
-        /// </summary>
-        /// <param name="node">The MethodCallExpression.</param>
-        /// <returns>The MethodCallExpression.</returns>
-        protected override Expression VisitMethodCall(MethodCallExpression node)
-        {
-            WriteHeader(node);
-            using (new Indentation(this))
-            {
-                WriteLine("Method={0}", node.Method.Name);
-                WriteLine("Arguments:");
-                using (new Indentation(this))
-                {
-                    foreach (var arg in node.Arguments)
-                    {
-                        Visit(arg);
-                    }
-                }
-            }
-            return node;
-        }
-
         /// <summary>
         /// Visits a MemberAssignment.
         /// </summary>
@@ -214,63 +200,64 @@ protected override MemberBinding VisitMemberBinding(MemberBinding node)
         }
 
         /// <summary>
-        /// Visits a MemberMemberBinding.
+        /// Visits a MemberBinding list.
         /// </summary>
-        /// <param name="node">The MemberMemberBinding.</param>
-        /// <returns>The MemberMemberBinding.</returns>
-        protected override MemberMemberBinding VisitMemberMemberBinding(MemberMemberBinding node)
+        /// <param name="nodes">The MemberBinding list.</param>
+        /// <returns>The MemberBinding list.</returns>
+        protected override IEnumerable<MemberBinding> VisitMemberBindingList(ReadOnlyCollection<MemberBinding> nodes)
         {
             throw new NotImplementedException();
         }
 
         /// <summary>
-        /// Visits a MemberListBinding.
+        /// Visits a MemberInitExpression.
         /// </summary>
-        /// <param name="node">The MemberListBinding.</param>
-        /// <returns>The MemberListBinding.</returns>
-        protected override MemberListBinding VisitMemberListBinding(MemberListBinding node)
+        /// <param name="node">The MemberInitExpression.</param>
+        /// <returns>The MemberInitExpression.</returns>
+        protected override Expression VisitMemberInit(MemberInitExpression node)
         {
             throw new NotImplementedException();
         }
 
         /// <summary>
-        /// Visits a MemberBinding list.
+        /// Visits a MemberListBinding.
         /// </summary>
-        /// <param name="nodes">The MemberBinding list.</param>
-        /// <returns>The MemberBinding list.</returns>
-        protected override IEnumerable<MemberBinding> VisitMemberBindingList(ReadOnlyCollection<MemberBinding> nodes)
+        /// <param name="node">The MemberListBinding.</param>
+        /// <returns>The MemberListBinding.</returns>
+        protected override MemberListBinding VisitMemberListBinding(MemberListBinding node)
         {
             throw new NotImplementedException();
         }
 
         /// <summary>
-        /// Visits an ElementInit list.
+        /// Visits a MemberMemberBinding.
         /// </summary>
-        /// <param name="nodes">The ElementInit list.</param>
-        /// <returns>The ElementInit list.</returns>
-        protected override IEnumerable<ElementInit> VisitElementInitializerList(
-            ReadOnlyCollection<ElementInit> nodes)
+        /// <param name="node">The MemberMemberBinding.</param>
+        /// <returns>The MemberMemberBinding.</returns>
+        protected override MemberMemberBinding VisitMemberMemberBinding(MemberMemberBinding node)
         {
             throw new NotImplementedException();
         }
 
         /// <summary>
-        /// Visits a LambdaExpression.
+        /// Visits a MethodCallExpression.
         /// </summary>
-        /// <param name="node">The LambdaExpression.</param>
-        /// <returns>The LambdaExpression.</returns>
-        protected override Expression VisitLambda(LambdaExpression node)
+        /// <param name="node">The MethodCallExpression.</param>
+        /// <returns>The MethodCallExpression.</returns>
+        protected override Expression VisitMethodCall(MethodCallExpression node)
         {
             WriteHeader(node);
             using (new Indentation(this))
             {
-                WriteLine("Parameters:");
-                foreach (var parameter in node.Parameters)
+                WriteLine("Method={0}", node.Method.Name);
+                WriteLine("Arguments:");
+                using (new Indentation(this))
                 {
-                    VisitIndented(parameter);
+                    foreach (var arg in node.Arguments)
+                    {
+                        Visit(arg);
+                    }
                 }
-                WriteLine("Body:");
-                VisitIndented(node.Body);
             }
             return node;
         }
@@ -295,43 +282,56 @@ protected override NewExpression VisitNew(NewExpression node)
         }
 
         /// <summary>
-        /// Visits a MemberInitExpression.
+        /// Visits a NewArrayExpression.
         /// </summary>
-        /// <param name="node">The MemberInitExpression.</param>
-        /// <returns>The MemberInitExpression.</returns>
-        protected override Expression VisitMemberInit(MemberInitExpression node)
+        /// <param name="node">The NewArrayExpression.</param>
+        /// <returns>The NewArrayExpression.</returns>
+        protected override Expression VisitNewArray(NewArrayExpression node)
         {
             throw new NotImplementedException();
         }
 
         /// <summary>
-        /// Visits a ListInitExpression.
+        /// Visits a ParameterExpression.
         /// </summary>
-        /// <param name="node">The ListInitExpression.</param>
-        /// <returns>The ListInitExpression.</returns>
-        protected override Expression VisitListInit(ListInitExpression node)
+        /// <param name="node">The ParameterExpression.</param>
+        /// <returns>The ParameterExpression.</returns>
+        protected override Expression VisitParameter(ParameterExpression node)
         {
-            throw new NotImplementedException();
+            WriteHeader(node);
+            using (new Indentation(this))
+            {
+                WriteLine("Name={0}", node.Name);
+                WriteLine("Type={0}", FriendlyClassName(node.Type));
+            }
+            return node;
         }
 
         /// <summary>
-        /// Visits a NewArrayExpression.
+        /// Visits a TypeBinaryExpression.
         /// </summary>
-        /// <param name="node">The NewArrayExpression.</param>
-        /// <returns>The NewArrayExpression.</returns>
-        protected override Expression VisitNewArray(NewArrayExpression node)
+        /// <param name="node">The TypeBinaryExpression.</param>
+        /// <returns>The TypeBinaryExpression.</returns>
+        protected override Expression VisitTypeBinary(TypeBinaryExpression node)
         {
             throw new NotImplementedException();
         }
 
         /// <summary>
-        /// Visits an InvocationExpression.
+        /// Visits a UnaryExpression.
         /// </summary>
-        /// <param name="node">The InvocationExpression.</param>
-        /// <returns>The InvocationExpression.</returns>
-        protected override Expression VisitInvocation(InvocationExpression node)
+        /// <param name="node">The UnaryExpression.</param>
+        /// <returns>The UnaryExpression.</returns>
+        protected override Expression VisitUnary(UnaryExpression node)
         {
-            throw new NotImplementedException();
+            WriteHeader(node);
+            using (new Indentation(this))
+            {
+                WriteLine("Method={0}", node.Method == null ? "null" : node.Method.Name);
+                WriteLine("Operand:");
+                VisitIndented(node.Operand);
+            }
+            return node;
         }
 
         // private methods