Skip to content

Commit b640ef1

Browse files
authored
Add comment about 6 params (#6125)
1 parent c5c2bd0 commit b640ef1

File tree

1 file changed

+4
-0
lines changed

1 file changed

+4
-0
lines changed

vm/src/function/argument.rs

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -55,6 +55,10 @@ into_func_args_from_tuple!((v1, T1), (v2, T2), (v3, T3));
5555
into_func_args_from_tuple!((v1, T1), (v2, T2), (v3, T3), (v4, T4));
5656
into_func_args_from_tuple!((v1, T1), (v2, T2), (v3, T3), (v4, T4), (v5, T5));
5757
into_func_args_from_tuple!((v1, T1), (v2, T2), (v3, T3), (v4, T4), (v5, T5), (v6, T6));
58+
// We currently allows only 6 unnamed positional arguments.
59+
// Please use `#[derive(FromArgs)]` and a struct for more complex argument parsing.
60+
// The number of limitation came from:
61+
// https://rust-lang.github.io/rust-clippy/master/index.html#too_many_arguments
5862

5963
/// The `FuncArgs` struct is one of the most used structs then creating
6064
/// a rust function that can be called from python. It holds both positional

0 commit comments

Comments
 (0)