Skip to content

Commit

Permalink
Rename 'delims' to 'delimpv' for clarity
Browse files Browse the repository at this point in the history
  • Loading branch information
leonerd committed Sep 15, 2023
1 parent f7144bc commit 02d9e57
Showing 1 changed file with 4 additions and 4 deletions.
8 changes: 4 additions & 4 deletions doop.c
Original file line number Diff line number Diff line change
Expand Up @@ -662,7 +662,7 @@ Perl_do_join(pTHX_ SV *sv, SV *delim, SV **mark, SV **sp)
I32 items = sp - mark;
STRLEN len;
STRLEN delimlen;
const char * delims = SvPV_const(delim, delimlen);
const char * delimpv = SvPV_const(delim, delimlen);

PERL_ARGS_ASSERT_DO_JOIN;

Expand Down Expand Up @@ -708,11 +708,11 @@ Perl_do_join(pTHX_ SV *sv, SV *delim, SV **mark, SV **sp)
/* Take a copy in case delim SV is a tied SV with a
* self-modifying FETCH [GH #21458]
*/
delims = savepvn(delims, delimlen);
SAVEFREEPV(delims);
delimpv = savepvn(delimpv, delimlen);
SAVEFREEPV(delimpv);
delim_copied = true;
}
sv_catpvn_flags(sv,delims,delimlen,delimflag);
sv_catpvn_flags(sv,delimpv,delimlen,delimflag);
s = SvPV_const(*mark,len);
sv_catpvn_flags(sv,s,len,
DO_UTF8(*mark) ? SV_CATUTF8 : SV_CATBYTES);
Expand Down

0 comments on commit 02d9e57

Please sign in to comment.