summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorPascal Rigaux <pixel@mandriva.com>2004-10-13 04:26:48 +0000
committerPascal Rigaux <pixel@mandriva.com>2004-10-13 04:26:48 +0000
commite6798126b96281772d88475d1addfe53f30e42d2 (patch)
treec99c6d9b5c871964d55476425377fc8f34eef7bd
parent01f4e8c175d99ff78990a53100b1036d50d0211c (diff)
downloadperl_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.ml2
-rw-r--r--perl_checker.src/test/return_value.t2
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