diff options
author | Pascal Rigaux <pixel@mandriva.com> | 2004-08-12 04:46:01 +0000 |
---|---|---|
committer | Pascal Rigaux <pixel@mandriva.com> | 2004-08-12 04:46:01 +0000 |
commit | c927e68bfd5e070d666642de786a971b9f0330e9 (patch) | |
tree | 012c5276c3678c50f0a02b43a4deba27081b5df5 /perl_checker.src/parser_helper.ml | |
parent | 230f4ddd8cbac8d2be387810de58248900004243 (diff) | |
download | perl_checker-c927e68bfd5e070d666642de786a971b9f0330e9.tar perl_checker-c927e68bfd5e070d666642de786a971b9f0330e9.tar.gz perl_checker-c927e68bfd5e070d666642de786a971b9f0330e9.tar.bz2 perl_checker-c927e68bfd5e070d666642de786a971b9f0330e9.tar.xz perl_checker-c927e68bfd5e070d666642de786a971b9f0330e9.zip |
you can replace "grep { !member($_, ...) } @l" with "difference2([ @l ], [ ... ])"
Diffstat (limited to 'perl_checker.src/parser_helper.ml')
-rw-r--r-- | perl_checker.src/parser_helper.ml | 9 |
1 files changed, 8 insertions, 1 deletions
diff --git a/perl_checker.src/parser_helper.ml b/perl_checker.src/parser_helper.ml index 3700f2f..8d238ed 100644 --- a/perl_checker.src/parser_helper.ml +++ b/perl_checker.src/parser_helper.ml @@ -883,7 +883,14 @@ let call_raw force_non_builtin_func (e, para) = warn_rule "you can replace \"map { if_(..., $_) }\" with \"grep { ... }\"" | _ -> check_anonymous_block f para) - | "grep" + | "grep" -> + (match para with + | [ Anonymous_sub(None, Block [ List [ Call_op("not", [ Call(Deref(I_func, Ident(None, "member", _)), [ List(Deref(I_scalar, Ident(None, "_", _)) :: _) ]) ], _) ] ], _); _ ] -> + warn_rule "you can replace \"grep { !member($_, ...) } @l\" with \"difference2([ @l ], [ ... ])\"" + | [ Anonymous_sub(None, Block [ List [ Call(Deref(I_func, Ident(None, "member", _)), [ List(Deref(I_scalar, Ident(None, "_", _)) :: _) ]) ] ], _); _ ] -> + warn_rule "you can replace \"grep { member($_, ...) } @l\" with \"intersection([ @l ], [ ... ])\"" + | _ -> check_anonymous_block f para) + | "grep_index" | "map_index" | "partition" | "uniq_" | "find" | "any" | "every" |