StackRduino
diff --git a/‎py/objfun.c‎
Lines changed: 8 additions & 7 deletions b/‎py/objfun.c‎
Lines changed: 8 additions & 7 deletions
@@ -30,26 +30,27 @@ STATIC void check_nargs(mp_obj_fun_native_t *self, int n_args, int n_kw) {
 }
 
 void mp_check_nargs(int n_args, machine_uint_t n_args_min, machine_uint_t n_args_max, int n_kw, bool is_kw) {
+    // TODO maybe take the function name as an argument so we can print nicer error messages
+
     if (n_kw && !is_kw) {
-        nlr_raise(mp_obj_new_exception_msg(&mp_type_TypeError,
-                                          "function does not take keyword arguments"));
+        nlr_raise(mp_obj_new_exception_msg(&mp_type_TypeError, "function does not take keyword arguments"));
     }
 
     if (n_args_min == n_args_max) {
         if (n_args != n_args_min) {
             nlr_raise(mp_obj_new_exception_msg_varg(&mp_type_TypeError,
-                                                     "function takes %d positional arguments but %d were given",
-                                                     n_args_min, n_args));
+                                                    "function takes %d positional arguments but %d were given",
+                                                    n_args_min, n_args));
         }
     } else {
         if (n_args < n_args_min) {
             nlr_raise(mp_obj_new_exception_msg_varg(&mp_type_TypeError,
-                                                    "<fun name>() missing %d required positional arguments: <list of names of params>",
+                                                    "function missing %d required positional arguments",
                                                     n_args_min - n_args));
         } else if (n_args > n_args_max) {
             nlr_raise(mp_obj_new_exception_msg_varg(&mp_type_TypeError,
-                                                     "<fun name> expected at most %d arguments, got %d",
-                                                     n_args_max, n_args));
+                                                    "function expected at most %d arguments, got %d",
+                                                    n_args_max, n_args));
         }
     }
 }
Original file line number	Diff line number	Diff line change
`@@ -30,26 +30,27 @@ STATIC void check_nargs(mp_obj_fun_native_t *self, int n_args, int n_kw) {`
`30`	`30`	`}`
`31`	`31`
`32`	`32`	`void mp_check_nargs(int n_args, machine_uint_t n_args_min, machine_uint_t n_args_max, int n_kw, bool is_kw) {`
	`33`	`+ // TODO maybe take the function name as an argument so we can print nicer error messages`
	`34`	`+`
`33`	`35`	`if (n_kw && !is_kw) {`
`34`		`- nlr_raise(mp_obj_new_exception_msg(&mp_type_TypeError,`
`35`		`- "function does not take keyword arguments"));`
	`36`	`+ nlr_raise(mp_obj_new_exception_msg(&mp_type_TypeError, "function does not take keyword arguments"));`
`36`	`37`	`}`
`37`	`38`
`38`	`39`	`if (n_args_min == n_args_max) {`
`39`	`40`	`if (n_args != n_args_min) {`
`40`	`41`	`nlr_raise(mp_obj_new_exception_msg_varg(&mp_type_TypeError,`
`41`		`- "function takes %d positional arguments but %d were given",`
`42`		`- n_args_min, n_args));`
	`42`	`+ "function takes %d positional arguments but %d were given",`
	`43`	`+ n_args_min, n_args));`
`43`	`44`	`}`
`44`	`45`	`} else {`
`45`	`46`	`if (n_args < n_args_min) {`
`46`	`47`	`nlr_raise(mp_obj_new_exception_msg_varg(&mp_type_TypeError,`
`47`		`- "<fun name>() missing %d required positional arguments: <list of names of params>",`
	`48`	`+ "function missing %d required positional arguments",`
`48`	`49`	`n_args_min - n_args));`
`49`	`50`	`} else if (n_args > n_args_max) {`
`50`	`51`	`nlr_raise(mp_obj_new_exception_msg_varg(&mp_type_TypeError,`
`51`		`- "<fun name> expected at most %d arguments, got %d",`
`52`		`- n_args_max, n_args));`
	`52`	`+ "function expected at most %d arguments, got %d",`
	`53`	`+ n_args_max, n_args));`
`53`	`54`	`}`
`54`	`55`	`}`
`55`	`56`	`}`