diff options
| author | Pascal Rigaux <pixel@mandriva.com> | 2004-04-28 12:55:44 +0000 |
|---|---|---|
| committer | Pascal Rigaux <pixel@mandriva.com> | 2004-04-28 12:55:44 +0000 |
| commit | 47e49317bcb377a94fdf14c8172710c962460e99 (patch) | |
| tree | 01872c5f0c624b3beb390dd64d83c0bc2bb2becf /perl_checker.src/parser_helper.ml | |
| parent | f1e322310f4ca8240b3da447d18b716dec7f7262 (diff) | |
| download | perl-MDK-Common-47e49317bcb377a94fdf14c8172710c962460e99.tar perl-MDK-Common-47e49317bcb377a94fdf14c8172710c962460e99.tar.gz perl-MDK-Common-47e49317bcb377a94fdf14c8172710c962460e99.tar.bz2 perl-MDK-Common-47e49317bcb377a94fdf14c8172710c962460e99.tar.xz perl-MDK-Common-47e49317bcb377a94fdf14c8172710c962460e99.zip | |
various detection of join() misuse
Diffstat (limited to 'perl_checker.src/parser_helper.ml')
| -rw-r--r-- | perl_checker.src/parser_helper.ml | 9 |
1 files changed, 9 insertions, 0 deletions
diff --git a/perl_checker.src/parser_helper.ml b/perl_checker.src/parser_helper.ml index fc578a1..113d57b 100644 --- a/perl_checker.src/parser_helper.ml +++ b/perl_checker.src/parser_helper.ml @@ -832,6 +832,15 @@ let call_raw force_non_builtin_func (e, para) = | [ Ident(None, s, pos) ] -> Some [ Raw_string(s, pos) ] | _ -> None) + + | "join" -> + (match un_parenthesize_full_l para with + | e :: _ when not (is_a_scalar e) -> warn_rule "first argument of join() must be a scalar"; + | [_] -> warn_rule "not enough parameters" + | [_; e] when is_a_scalar e -> warn_rule "join('...', $foo) is the same as $foo" + | _ -> ()); + None + | "last" | "next" | "redo" when not force_non_builtin_func -> (match para with | [ Ident(None, s, pos) ] -> Some [ Raw_string(s, pos) ] |