diff options
author | Gurusamy Sarathy <gsar@cpan.org> | 1999-06-27 13:31:11 +0000 |
---|---|---|
committer | Gurusamy Sarathy <gsar@cpan.org> | 1999-06-27 13:31:11 +0000 |
commit | d0334bed0b5f8315518daa5bbcd832e006b78148 (patch) | |
tree | e2602535c211eed3593d449a8ac7cae53f40537c | |
parent | c9fdb9e966a1ba8953c235de2e289c17cdd8c75f (diff) | |
download | perl-d0334bed0b5f8315518daa5bbcd832e006b78148.tar.gz |
fix indents
p4raw-id: //depot/perl@3551
-rw-r--r-- | op.c | 36 |
1 files changed, 20 insertions, 16 deletions
@@ -4993,23 +4993,27 @@ OP * Perl_ck_defined(pTHX_ OP *o) /* 19990527 MJD */ { dTHR; - if (ckWARN(WARN_DEPRECATED) && (o->op_flags & OPf_KIDS)) { - switch (cUNOPo->op_first->op_type) { - case OP_RV2AV: - case OP_PADAV: - case OP_AASSIGN: /* Is this a good idea? */ - Perl_warner(aTHX_ WARN_DEPRECATED, "defined(@array) is deprecated (and not really meaningful)"); - Perl_warner(aTHX_ WARN_DEPRECATED, "(Maybe you should just omit the defined()?)\n"); + if ((o->op_flags & OPf_KIDS) && ckWARN(WARN_DEPRECATED)) { + switch (cUNOPo->op_first->op_type) { + case OP_RV2AV: + case OP_PADAV: + case OP_AASSIGN: /* Is this a good idea? */ + Perl_warner(aTHX_ WARN_DEPRECATED, + "defined(@array) is deprecated (and not really meaningful)"); + Perl_warner(aTHX_ WARN_DEPRECATED, + "(Maybe you should just omit the defined()?)\n"); break; - case OP_RV2HV: - case OP_PADHV: - Perl_warner(aTHX_ WARN_DEPRECATED, "defined(%hash) is deprecated (and not really meaningful)"); - Perl_warner(aTHX_ WARN_DEPRECATED, "(Maybe you should just omit the defined()?)\n"); - break; - default: - /* no warning */ - break; - } + case OP_RV2HV: + case OP_PADHV: + Perl_warner(aTHX_ WARN_DEPRECATED, + "defined(%hash) is deprecated (and not really meaningful)"); + Perl_warner(aTHX_ WARN_DEPRECATED, + "(Maybe you should just omit the defined()?)\n"); + break; + default: + /* no warning */ + break; + } } return ck_rfun(o); } |