Code cleanup

Author: kalgiz

Engine/Helper.cs

@@ -631,30 +631,20 @@ public bool PositionalParameterUsed(CommandAst cmdAst, bool moreThanTwoPositiona
 
             // Because of the way we count, we will also count the cmdlet as an argument so we have to -1
             int argumentsWithoutProcedingParameters = -1;
-            bool parameterPreceding = false;
 
-            foreach (CommandElementAst ceAst in cmdAst.CommandElements)
-            {
-                var cmdParamAst = ceAst as CommandParameterAst;
-                if (cmdParamAst != null)
-                {
-                    parameterPreceding = true;
-                } else
+            var commandElementCollection = cmdAst.CommandElements;
+            for (int i = 0; i < commandElementCollection.Count(); i++) {
+                if (!(commandElementCollection[i] is CommandParameterAst) && (i ==0 || !(commandElementCollection[i-1] is CommandParameterAst)))
                 {
-                    if (!parameterPreceding)
-                    {
-                        argumentsWithoutProcedingParameters += 1;
-                    }
-                    parameterPreceding = false;
+                    argumentsWithoutProcedingParameters++;
                 }
             }
 
             // if not the first element in a pipeline, increase the number of arguments by 1
             PipelineAst parent = cmdAst.Parent as PipelineAst;
-
             if (parent != null && parent.PipelineElements.Count > 1 && parent.PipelineElements[0] != cmdAst)
             {
-                argumentsWithoutProcedingParameters += 1;
+                argumentsWithoutProcedingParameters++;
             }
 
             return moreThanTwoPositional ? argumentsWithoutProcedingParameters > 2 : argumentsWithoutProcedingParameters > 0;