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
commit162f86dcdf252a8ba4b43c8f11ca248a58d5c56a (patch)
tree5c12c4e3aa4ec8d1640d873d9b5a4f22f9ca89a0
parentfac5095d6f0dce57451e0b0b3f244187b407407e (diff)
downloadperl-MDK-Common-162f86dcdf252a8ba4b43c8f11ca248a58d5c56a.tar
perl-MDK-Common-162f86dcdf252a8ba4b43c8f11ca248a58d5c56a.tar.gz
perl-MDK-Common-162f86dcdf252a8ba4b43c8f11ca248a58d5c56a.tar.bz2
perl-MDK-Common-162f86dcdf252a8ba4b43c8f11ca248a58d5c56a.tar.xz
perl-MDK-Common-162f86dcdf252a8ba4b43c8f11ca248a58d5c56a.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