Browse Source

Create a function to parse the argument names

Frederic Torres 11 years ago
parent
commit
d17034fe07
1 changed files with 4 additions and 3 deletions
  1. 4 3
      Jint/Native/Function/FunctionConstructor.cs

+ 4 - 3
Jint/Native/Function/FunctionConstructor.cs

@@ -45,8 +45,10 @@ namespace Jint.Native.Function
             return Construct(arguments);
             return Construct(arguments);
         }
         }
 
 
-        private string[] TrimParameterNames(string[] values)
+        private string[] ParseArgumentNames(string parameterDeclaration)
         {
         {
+            string[] values = parameterDeclaration.Split(new[] { ',' }, StringSplitOptions.RemoveEmptyEntries);
+
             var newValues = new string[values.Length];
             var newValues = new string[values.Length];
             for (var i = 0; i < values.Length; i++)
             for (var i = 0; i < values.Length; i++)
             {
             {
@@ -79,8 +81,7 @@ namespace Jint.Native.Function
             }
             }
             body = TypeConverter.ToString(body);
             body = TypeConverter.ToString(body);
             
             
-            // todo: ensure parsable as parameter list
-            var parameters = this.TrimParameterNames(p.Split(new [] {','}, StringSplitOptions.RemoveEmptyEntries));
+            var parameters = this.ParseArgumentNames(p);
             var parser = new JavaScriptParser();
             var parser = new JavaScriptParser();
             FunctionExpression function;
             FunctionExpression function;
             try
             try