diff options
| author | Father Chrysostomos <sprout@cpan.org> | 2012-05-12 14:25:44 -0700 |
|---|---|---|
| committer | Father Chrysostomos <sprout@cpan.org> | 2012-05-29 09:36:25 -0700 |
| commit | 89c5c07ef37776cd5a06bc62eaca97cb2091ad3f (patch) | |
| tree | ebaafe25f710be6dd0a904abd828a50278b7375f /regen | |
| parent | 498a02d80cfca0073bd62ad27aac793a0d5e785c (diff) | |
| download | perl-89c5c07ef37776cd5a06bc62eaca97cb2091ad3f.tar.gz | |
Make undef use ck_fun and OA_SCALARREF
In regen/opcodes, we have some operators that use ck_fun and have R
for the argument. And there are some that use ck_lfun and have S for
the argument.
These both amount to more or less the same thing.
ck_fun/R goes through the OA_SCALARREF case in ck_fun, which calls
op_lvalue(scalar()) on the child op.
ck_lfun/S goes through the OA_SCALAR case in ck_fun, which calls
scalar(), and then ck_lfun sees to it that op_lvalue is called.
The only real difference is that the OA_SCALAR case makes sure there
are not too many arguments.
Since both core_prototype and pp_coreargs need special cases to deal
with OA_SCALAR that is really ‘supposed’ to be OA_SCALARREF, it
becomes simpler to add &CORE::undef if undef uses R. In that case,
we also have to put the argument-checking in the OA_SCALARREF, but we
make it conditional on the op being an undef op, for the sake of doing
one thing at a time. (This is a bit of a mess; see ticket #96006.)
Diffstat (limited to 'regen')
| -rw-r--r-- | regen/opcodes | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/regen/opcodes b/regen/opcodes index 0beba6a90d..592ecebde2 100644 --- a/regen/opcodes +++ b/regen/opcodes @@ -103,7 +103,7 @@ schop scalar chop ck_null stu% S? chomp chomp ck_spair mTs% L schomp scalar chomp ck_null sTu% S? defined defined operator ck_defined isu% S? -undef undef operator ck_lfun s% S? +undef undef operator ck_fun s% R? study study ck_fun su% S? pos match position ck_lfun stu% S? |