sv.c:S_curse: remove unnecessary null check

If an SV has the OBJECT flag on but no STASH, it will cause crashes
elsewhere.  So there has to be a stash here.  I only put the null
check there in 8c34e50dc because I was copying what StashHANDLER
used to do.  8c34e50dc removed the use of StashHANDLER (which calls
gv_handler, which has a null check), replacing it with a different
caching mechanism inlined into S_curse.

1 files changed, 2 insertions, 1 deletions

diff --git a/sv.c b/sv.c
index 5ecfbeb50b..9aa2224c42 100644
--- a/sv.c
+++ b/sv.c
@@ -6332,7 +6332,8 @@ S_curse(pTHX_ SV * const sv, const bool check_refcnt) {
 	dSP;
 	HV* stash;
 	do {
-	  if ((stash = SvSTASH(sv)) && HvNAME(stash)) {
+	  stash = SvSTASH(sv);
+	  if (HvNAME(stash)) {
 	    CV* destructor = NULL;
 	    assert(SvTYPE(stash) == SVt_PVHV);
 	    if (!SvOBJECT(stash)) destructor = (CV *)SvSTASH(stash);