diff options
author | Pascal Rigaux <pixel@mandriva.com> | 2004-10-13 04:26:48 +0000 |
---|---|---|
committer | Pascal Rigaux <pixel@mandriva.com> | 2004-10-13 04:26:48 +0000 |
commit | e6798126b96281772d88475d1addfe53f30e42d2 (patch) | |
tree | c99c6d9b5c871964d55476425377fc8f34eef7bd | |
parent | 01f4e8c175d99ff78990a53100b1036d50d0211c (diff) | |
download | perl_checker-e6798126b96281772d88475d1addfe53f30e42d2.tar perl_checker-e6798126b96281772d88475d1addfe53f30e42d2.tar.gz perl_checker-e6798126b96281772d88475d1addfe53f30e42d2.tar.bz2 perl_checker-e6798126b96281772d88475d1addfe53f30e42d2.tar.xz perl_checker-e6798126b96281772d88475d1addfe53f30e42d2.zip |
disallow the use of the return value of push and unshift
-rw-r--r-- | perl_checker.src/parser_helper.ml | 2 | ||||
-rw-r--r-- | perl_checker.src/test/return_value.t | 2 |
2 files changed, 3 insertions, 1 deletions
diff --git a/perl_checker.src/parser_helper.ml b/perl_checker.src/parser_helper.ml index 8d238ed..0be9353 100644 --- a/perl_checker.src/parser_helper.ml +++ b/perl_checker.src/parser_helper.ml @@ -362,7 +362,7 @@ let function_to_context word_alone = function | "each_index" -> M_none | "N" | "N_" -> M_string - | "chop" | "chomp" -> M_none + | "chop" | "chomp" | "push" | "unshift" -> M_none | "hex" | "length" | "time" | "fork" | "getppid" -> M_int | "eof" | "wantarray" -> M_int | "stat" | "lstat" -> M_list diff --git a/perl_checker.src/test/return_value.t b/perl_checker.src/test/return_value.t index fd8eafd..cae8652 100644 --- a/perl_checker.src/test/return_value.t +++ b/perl_checker.src/test/return_value.t @@ -15,3 +15,5 @@ map { xxx($_) } @l; if you don't use the return value, use $xxx = chomp; () context not accepted here context () is not compatible with context scalar +$xxx = push @l, 1 () context not accepted here + context () is not compatible with context scalar |