Skip to content

Commit 2d8768c

Browse files
committed
Remove redundant assert() statements
These assertions have been redundant since 2463f19, July 2024.
1 parent 9df4510 commit 2d8768c

File tree

3 files changed

+0
-22
lines changed

3 files changed

+0
-22
lines changed

av.c

Lines changed: 0 additions & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -25,7 +25,6 @@ Perl_av_reify(pTHX_ AV *av)
2525
SSize_t key;
2626

2727
PERL_ARGS_ASSERT_AV_REIFY;
28-
assert(SvTYPE(av) == SVt_PVAV);
2928

3029
if (AvREAL(av))
3130
return;
@@ -68,7 +67,6 @@ Perl_av_extend(pTHX_ AV *av, SSize_t key)
6867
MAGIC *mg;
6968

7069
PERL_ARGS_ASSERT_AV_EXTEND;
71-
assert(SvTYPE(av) == SVt_PVAV);
7270

7371
mg = SvTIED_mg((const SV *)av, PERL_MAGIC_tied);
7472
if (mg) {
@@ -272,7 +270,6 @@ Perl_av_fetch(pTHX_ AV *av, SSize_t key, I32 lval)
272270
SSize_t size;
273271

274272
PERL_ARGS_ASSERT_AV_FETCH;
275-
assert(SvTYPE(av) == SVt_PVAV);
276273

277274
if (UNLIKELY(SvRMAGICAL(av))) {
278275
const MAGIC * const tied_magic
@@ -342,7 +339,6 @@ Perl_av_store(pTHX_ AV *av, SSize_t key, SV *val)
342339
SV** ary;
343340

344341
PERL_ARGS_ASSERT_AV_STORE;
345-
assert(SvTYPE(av) == SVt_PVAV);
346342

347343
/* S_regclass relies on being able to pass in a NULL sv
348344
(unicode_alternate may be NULL).
@@ -637,7 +633,6 @@ Perl_av_clear(pTHX_ AV *av)
637633
SSize_t orig_ix = 0;
638634

639635
PERL_ARGS_ASSERT_AV_CLEAR;
640-
assert(SvTYPE(av) == SVt_PVAV);
641636

642637
#ifdef DEBUGGING
643638
if (SvREFCNT(av) == 0) {
@@ -711,7 +706,6 @@ Perl_av_undef(pTHX_ AV *av)
711706
SSize_t orig_ix = PL_tmps_ix; /* silence bogus warning about possible uninitialized use */
712707

713708
PERL_ARGS_ASSERT_AV_UNDEF;
714-
assert(SvTYPE(av) == SVt_PVAV);
715709

716710
/* Give any tie a chance to cleanup first */
717711
if (SvTIED_mg((const SV *)av, PERL_MAGIC_tied))
@@ -793,7 +787,6 @@ Perl_av_push(pTHX_ AV *av, SV *val)
793787
MAGIC *mg;
794788

795789
PERL_ARGS_ASSERT_AV_PUSH;
796-
assert(SvTYPE(av) == SVt_PVAV);
797790

798791
if (SvREADONLY(av))
799792
croak_no_modify();
@@ -825,7 +818,6 @@ Perl_av_pop(pTHX_ AV *av)
825818
MAGIC* mg;
826819

827820
PERL_ARGS_ASSERT_AV_POP;
828-
assert(SvTYPE(av) == SVt_PVAV);
829821

830822
if (SvREADONLY(av))
831823
croak_no_modify();
@@ -884,7 +876,6 @@ Perl_av_unshift(pTHX_ AV *av, SSize_t num)
884876
MAGIC* mg;
885877

886878
PERL_ARGS_ASSERT_AV_UNSHIFT;
887-
assert(SvTYPE(av) == SVt_PVAV);
888879

889880
if (SvREADONLY(av))
890881
croak_no_modify();
@@ -951,7 +942,6 @@ Perl_av_shift(pTHX_ AV *av)
951942
MAGIC* mg;
952943

953944
PERL_ARGS_ASSERT_AV_SHIFT;
954-
assert(SvTYPE(av) == SVt_PVAV);
955945

956946
if (SvREADONLY(av))
957947
croak_no_modify();
@@ -1027,7 +1017,6 @@ Perl_av_fill(pTHX_ AV *av, SSize_t fill)
10271017
MAGIC *mg;
10281018

10291019
PERL_ARGS_ASSERT_AV_FILL;
1030-
assert(SvTYPE(av) == SVt_PVAV);
10311020

10321021
if (fill < 0)
10331022
fill = -1;
@@ -1080,7 +1069,6 @@ Perl_av_delete(pTHX_ AV *av, SSize_t key, I32 flags)
10801069
SV *sv;
10811070

10821071
PERL_ARGS_ASSERT_AV_DELETE;
1083-
assert(SvTYPE(av) == SVt_PVAV);
10841072

10851073
if (SvREADONLY(av))
10861074
croak_no_modify();
@@ -1155,7 +1143,6 @@ bool
11551143
Perl_av_exists(pTHX_ AV *av, SSize_t key)
11561144
{
11571145
PERL_ARGS_ASSERT_AV_EXISTS;
1158-
assert(SvTYPE(av) == SVt_PVAV);
11591146

11601147
if (SvRMAGICAL(av)) {
11611148
const MAGIC * const tied_magic
@@ -1213,7 +1200,6 @@ S_get_aux_mg(pTHX_ AV *av) {
12131200
MAGIC *mg;
12141201

12151202
PERL_ARGS_ASSERT_GET_AUX_MG;
1216-
assert(SvTYPE(av) == SVt_PVAV);
12171203

12181204
mg = mg_find((const SV *)av, PERL_MAGIC_arylen_p);
12191205

@@ -1232,7 +1218,6 @@ Perl_av_arylen_p(pTHX_ AV *av) {
12321218
MAGIC *const mg = get_aux_mg(av);
12331219

12341220
PERL_ARGS_ASSERT_AV_ARYLEN_P;
1235-
assert(SvTYPE(av) == SVt_PVAV);
12361221

12371222
return &(mg->mg_obj);
12381223
}
@@ -1242,7 +1227,6 @@ Perl_av_iter_p(pTHX_ AV *av) {
12421227
MAGIC *const mg = get_aux_mg(av);
12431228

12441229
PERL_ARGS_ASSERT_AV_ITER_P;
1245-
assert(SvTYPE(av) == SVt_PVAV);
12461230

12471231
if (sizeof(IV) == sizeof(SSize_t)) {
12481232
return (IV *)&(mg->mg_len);

inline.h

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -56,7 +56,6 @@ PERL_STATIC_INLINE Size_t
5656
Perl_av_count(pTHX_ AV *av)
5757
{
5858
PERL_ARGS_ASSERT_AV_COUNT;
59-
assert(SvTYPE(av) == SVt_PVAV);
6059

6160
return AvFILL(av) + 1;
6261
}
@@ -100,7 +99,6 @@ Perl_av_store_simple(pTHX_ AV *av, SSize_t key, SV *val)
10099
SV** ary;
101100

102101
PERL_ARGS_ASSERT_AV_STORE_SIMPLE;
103-
assert(SvTYPE(av) == SVt_PVAV);
104102
assert(!SvMAGICAL(av));
105103
assert(!SvREADONLY(av));
106104
assert(AvREAL(av));
@@ -143,7 +141,6 @@ PERL_STATIC_INLINE SV**
143141
Perl_av_fetch_simple(pTHX_ AV *av, SSize_t key, I32 lval)
144142
{
145143
PERL_ARGS_ASSERT_AV_FETCH_SIMPLE;
146-
assert(SvTYPE(av) == SVt_PVAV);
147144
assert(!SvMAGICAL(av));
148145
assert(!SvREADONLY(av));
149146
assert(AvREAL(av));
@@ -160,7 +157,6 @@ PERL_STATIC_INLINE void
160157
Perl_av_push_simple(pTHX_ AV *av, SV *val)
161158
{
162159
PERL_ARGS_ASSERT_AV_PUSH_SIMPLE;
163-
assert(SvTYPE(av) == SVt_PVAV);
164160
assert(!SvMAGICAL(av));
165161
assert(!SvREADONLY(av));
166162
assert(AvREAL(av));
@@ -3885,7 +3881,6 @@ Perl_clear_defarray_simple(pTHX_ AV *av)
38853881
{
38863882
PERL_ARGS_ASSERT_CLEAR_DEFARRAY_SIMPLE;
38873883

3888-
assert(SvTYPE(av) == SVt_PVAV);
38893884
assert(!SvREADONLY(av));
38903885
assert(!SvMAGICAL(av));
38913886
assert(SvREFCNT(av) == 1);

sv.c

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1297,7 +1297,6 @@ Perl_hv_auxalloc(pTHX_ HV *hv) {
12971297
void *new_body;
12981298

12991299
PERL_ARGS_ASSERT_HV_AUXALLOC;
1300-
assert(SvTYPE(hv) == SVt_PVHV);
13011300
assert(!HvHasAUX(hv));
13021301

13031302
#ifdef PURIFY

0 commit comments

Comments
 (0)