From: Nicholas Clark <nick@ccl4.org>
Date: Thu, 18 Jan 2007 15:48:13 +0000 (+0000)
Subject: Given that we now do the tests in S_visit() to avoid calling the
X-Git-Url: http://git.shadowcat.co.uk/gitweb/gitweb.cgi?a=commitdiff_plain;h=ea724faac0e9bd1598db5c1f1c5bd689c0be70fc;p=p5sagit%2Fp5-mst-13.2.git

Given that we now do the tests in S_visit() to avoid calling the
helper function when skippable, no need to retain those tests
inside the helper functions do_clean_objs() and do_clean_named_objs().

p4raw-id: //depot/perl@29872
---

diff --git a/sv.c b/sv.c
index 9ef3eef..b602dda 100644
--- a/sv.c
+++ b/sv.c
@@ -443,7 +443,8 @@ static void
 do_clean_objs(pTHX_ SV *ref)
 {
     dVAR;
-    if (SvROK(ref)) {
+    assert (SvROK(ref));
+    {
 	SV * const target = SvRV(ref);
 	if (SvOBJECT(target)) {
 	    DEBUG_D((PerlIO_printf(Perl_debug_log, "Cleaning object ref:\n "), sv_dump(ref)));
@@ -469,7 +470,8 @@ static void
 do_clean_named_objs(pTHX_ SV *sv)
 {
     dVAR;
-    if (SvTYPE(sv) == SVt_PVGV && isGV_with_GP(sv) && GvGP(sv)) {
+    assert(SvTYPE(sv) == SVt_PVGV);
+    if (isGV_with_GP(sv) && GvGP(sv)) {
 	if ((
 #ifdef PERL_DONT_CREATE_GVSV
 	     GvSV(sv) &&