{
I32 key;
+ assert(av);
+
if (AvREAL(av))
return;
#ifdef DEBUGGING
void
Perl_av_extend(pTHX_ AV *av, I32 key)
{
- MAGIC * const mg = SvTIED_mg((SV*)av, PERL_MAGIC_tied);
+ MAGIC *mg;
+
+ assert(av);
+
+ mg = SvTIED_mg((SV*)av, PERL_MAGIC_tied);
if (mg) {
dSP;
ENTER;
{
SV *sv;
- if (!av)
- return 0;
+ assert(av);
if (SvRMAGICAL(av)) {
const MAGIC * const tied_magic = mg_find((SV*)av, PERL_MAGIC_tied);
}
else if (AvREIFY(av)
&& (!AvARRAY(av)[key] /* eg. @_ could have freed elts */
- || SvTYPE(AvARRAY(av)[key]) == SVTYPEMASK)) {
+ || SvIS_FREED(AvARRAY(av)[key]))) {
AvARRAY(av)[key] = &PL_sv_undef; /* 1/2 reify */
goto emptyness;
}
{
SV** ary;
- if (!av)
- return 0;
+ assert(av);
+
if (!val)
val = &PL_sv_undef;
ary[key] = val;
if (SvSMAGICAL(av)) {
if (val != &PL_sv_undef) {
- MAGIC* mg = SvMAGIC(av);
+ const MAGIC* const mg = SvMAGIC(av);
sv_magic(val, (SV*)av, toLOWER(mg->mg_type), 0, key);
}
mg_set((SV*)av);
return av;
}
-AV *
-Perl_av_fake(pTHX_ register I32 size, register SV **strp)
-{
- register SV** ary;
- register AV * const av = (AV*)NEWSV(9,0);
-
- sv_upgrade((SV *)av, SVt_PVAV);
- Newx(ary,size+1,SV*);
- AvALLOC(av) = ary;
- Copy(strp,ary,size,SV*);
- AvREIFY_only(av);
- SvPV_set(av, (char*)ary);
- AvFILLp(av) = size - 1;
- AvMAX(av) = size - 1;
- while (size--) {
- assert (*strp);
- SvTEMP_off(*strp);
- strp++;
- }
- return av;
-}
-
/*
=for apidoc av_clear
{
register I32 key;
+ assert(av);
#ifdef DEBUGGING
if (SvREFCNT(av) == 0 && ckWARN_d(WARN_DEBUGGING)) {
Perl_warner(aTHX_ packWARN(WARN_DEBUGGING), "Attempt to clear deleted array");
}
#endif
- if (!av)
- return;
if (SvREADONLY(av))
Perl_croak(aTHX_ PL_no_modify);
void
Perl_av_undef(pTHX_ register AV *av)
{
- if (!av)
- return;
+ assert(av);
/* Give any tie a chance to cleanup first */
if (SvTIED_mg((SV*)av, PERL_MAGIC_tied))
{
dVAR;
MAGIC *mg;
- if (!av)
- return;
+ assert(av);
+
if (SvREADONLY(av))
Perl_croak(aTHX_ PL_no_modify);
SV *retval;
MAGIC* mg;
- if (!av)
- return &PL_sv_undef;
+ assert(av);
+
if (SvREADONLY(av))
Perl_croak(aTHX_ PL_no_modify);
if ((mg = SvTIED_mg((SV*)av, PERL_MAGIC_tied))) {
register I32 i;
MAGIC* mg;
- if (!av)
- return;
+ assert(av);
+
if (SvREADONLY(av))
Perl_croak(aTHX_ PL_no_modify);
SV *retval;
MAGIC* mg;
- if (!av)
- return &PL_sv_undef;
+ assert(av);
+
if (SvREADONLY(av))
Perl_croak(aTHX_ PL_no_modify);
if ((mg = SvTIED_mg((SV*)av, PERL_MAGIC_tied))) {
I32
Perl_av_len(pTHX_ register const AV *av)
{
+ assert(av);
return AvFILL(av);
}
{
dVAR;
MAGIC *mg;
- if (!av)
- Perl_croak(aTHX_ "panic: null array");
+
+ assert(av);
+
if (fill < 0)
fill = -1;
if ((mg = SvTIED_mg((SV*)av, PERL_MAGIC_tied))) {
}
if (fill <= AvMAX(av)) {
I32 key = AvFILLp(av);
- SV** ary = AvARRAY(av);
+ SV** const ary = AvARRAY(av);
if (AvREAL(av)) {
while (key > fill) {
{
SV *sv;
- if (!av)
- return Nullsv;
+ assert(av);
+
if (SvREADONLY(av))
Perl_croak(aTHX_ PL_no_modify);
if (adjust_index) {
key += AvFILL(av) + 1;
if (key < 0)
- return Nullsv;
+ return NULL;
}
}
svp = av_fetch(av, key, TRUE);
sv_unmagic(sv, PERL_MAGIC_tiedelem); /* No longer an element */
return sv;
}
- return Nullsv;
+ return NULL;
}
}
}
if (key < 0) {
key += AvFILL(av) + 1;
if (key < 0)
- return Nullsv;
+ return NULL;
}
if (key > AvFILLp(av))
- return Nullsv;
+ return NULL;
else {
if (!AvREAL(av) && AvREIFY(av))
av_reify(av);
}
if (flags & G_DISCARD) {
SvREFCNT_dec(sv);
- sv = Nullsv;
+ sv = NULL;
}
else if (AvREAL(av))
sv = sv_2mortal(sv);
bool
Perl_av_exists(pTHX_ AV *av, I32 key)
{
- if (!av)
- return FALSE;
-
+ assert(av);
if (SvRMAGICAL(av)) {
const MAGIC * const tied_magic = mg_find((SV*)av, PERL_MAGIC_tied);
if (tied_magic || mg_find((SV*)av, PERL_MAGIC_regdata)) {
- SV *sv = sv_newmortal();
+ SV * const sv = sv_newmortal();
MAGIC *mg;
/* Handle negative array indices 20020222 MJD */
if (key < 0) {
SV **
Perl_av_arylen_p(pTHX_ AV *av) {
dVAR;
- MAGIC *mg = mg_find((SV*)av, PERL_MAGIC_arylen_p);
+ MAGIC *mg;
+
+ assert(av);
+
+ mg = mg_find((SV*)av, PERL_MAGIC_arylen_p);
if (!mg) {
mg = sv_magicext((SV*)av, 0, PERL_MAGIC_arylen_p, &PL_vtbl_arylen_p,
0, 0);
-
- if (!mg) {
- Perl_die(aTHX_ "panic: av_arylen_p");
- }
+ assert(mg);
/* sv_magicext won't set this for us because we pass in a NULL obj */
mg->mg_flags |= MGf_REFCOUNTED;
}