pp_substr: checking SvOK() is superfluous after SvPV_force_nomg()

When replacing some characters in pp_substr, there should be no
need to check `SvOK(sv)`, since previous calls to
`SvPV_force_nomg(sv)` and (if needed) `sv_utf8_upgrade_nomg(sv);`
should always return a `sv` that is `SvOK`.

pp.c

@@ -3696,8 +3696,11 @@ PP_wrapped(pp_substr,
                 repl = (char*)bytes_to_utf8_free_me(
                                         (U8*)repl, &repl_len, &free_me);
             }
-            if (!SvOK(sv))
-                SvPVCLEAR(sv);
+
+            /* The earlier SvPV_force_nomg(sv, curlen) should have ensured
+             * that sv is SvOK, even if it wasn't beforehand. */
+            assert(SvOK(sv));
+
             sv_insert_flags(sv, byte_pos, byte_len, repl, repl_len, 0);
             Safefree(free_me);
         }