diff options
author | Robin Barker <rmbarker@cpan.org> | 2011-09-07 22:22:42 +0100 |
---|---|---|
committer | Father Chrysostomos <sprout@cpan.org> | 2011-09-08 08:24:31 -0700 |
commit | 14f0f125b45b58bbf2d9e275abf2fef1655f8a3e (patch) | |
tree | d1f498a7896c607ac46e1b049fbffa0848aa5bac | |
parent | f8ccc5c641d025d213119510fd38619b06273044 (diff) | |
download | perl-14f0f125b45b58bbf2d9e275abf2fef1655f8a3e.tar.gz |
remove unused variables and assignments
and silences some compiler warnings.
I do not understand the code in toke.c but the change aligns the code
with other uses of FUN0OP, it has no warnings and does not break any test.
-rw-r--r-- | op.c | 3 | ||||
-rw-r--r-- | pp.c | 3 | ||||
-rw-r--r-- | toke.c | 3 |
3 files changed, 3 insertions, 6 deletions
@@ -9292,10 +9292,9 @@ Perl_ck_entersub_args_core(pTHX_ OP *entersubop, GV *namegv, SV *protosv) PERL_ARGS_ASSERT_CK_ENTERSUB_ARGS_CORE; if (!opnum) { - OP *prev, *cvop; + OP *cvop; if (!aop->op_sibling) aop = cUNOPx(aop)->op_first; - prev = aop; aop = aop->op_sibling; for (cvop = aop; cvop->op_sibling; cvop = cvop->op_sibling) ; if (PL_madskills) while (aop != cvop && aop->op_type == OP_STUB) { @@ -155,12 +155,11 @@ static SV * S_rv2gv(pTHX_ SV *sv, const bool vivify_sv, const bool strict, const bool noinit) { - dSP; dVAR; + dVAR; if (!isGV(sv) || SvFAKE(sv)) SvGETMAGIC(sv); if (SvROK(sv)) { if (SvAMAGIC(sv)) { sv = amagic_deref_call(sv, to_gv_amg); - SPAGAIN; } wasref: sv = SvRV(sv); @@ -6927,8 +6927,7 @@ Perl_yylex(pTHX) case KEY___FILE__: FUN0OP( - pl_yylval.opval = (OP*)newSVOP(OP_CONST, 0, - newSVpv(CopFILE(PL_curcop),0)) + (OP*)newSVOP(OP_CONST, 0, newSVpv(CopFILE(PL_curcop),0)) ); case KEY___LINE__: |