Skip to content

Commit a29d6e1

Browse files
committed
WS Cleanup Stage #5 - break long define
For long define statements put the replacement part on their own line. Also wrap comments on long lines.
1 parent a16b12e commit a29d6e1

29 files changed

+426
-209
lines changed

XSUB.h

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -386,7 +386,8 @@ Rethrows a previously caught exception. See L<perlguts/"Exception Handling">.
386386
HS_CXT, __FILE__, "v" PERL_API_VERSION_STRING, XS_VERSION)
387387
#else
388388
/* should this be a #error? if you want both checked, you better supply XS_VERSION right? */
389-
# define XS_BOTHVERSION_SETXSUBFN_POPMARK_BOOTCHECK XS_APIVERSION_SETXSUBFN_POPMARK_BOOTCHECK
389+
# define XS_BOTHVERSION_SETXSUBFN_POPMARK_BOOTCHECK \
390+
XS_APIVERSION_SETXSUBFN_POPMARK_BOOTCHECK
390391
#endif
391392

392393
/* For a normal bootstrap without API or XS version checking.

cop.h

Lines changed: 16 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -549,7 +549,8 @@ typedef struct rcpv RCPV;
549549

550550
#define RCPVf_USE_STRLEN 1
551551
#define RCPVf_NO_COPY 2
552-
#define RCPVx(pv_arg) ((RCPV *)((pv_arg) - STRUCT_OFFSET(struct rcpv, pv)))
552+
#define RCPVx(pv_arg) \
553+
((RCPV *)((pv_arg) - STRUCT_OFFSET(struct rcpv, pv)))
553554
#define RCPV_REFCOUNT(pv) (RCPVx(pv)->refcount)
554555
#define RCPV_LEN(pv) (RCPVx(pv)->len)
555556

@@ -561,10 +562,12 @@ typedef struct rcpv RCPV;
561562
(CopFILE(c) \
562563
? gv_fetchfile(CopFILE(c)) : NULL)
563564

564-
# define CopFILE_set_x(c,pv) ((c)->cop_file = rcpv_new((pv),0,RCPVf_USE_STRLEN))
565+
# define CopFILE_set_x(c,pv) \
566+
((c)->cop_file = rcpv_new((pv),0,RCPVf_USE_STRLEN))
565567
# define CopFILE_setn_x(c,pv,l) ((c)->cop_file = rcpv_new((pv),(l),0))
566568
# define CopFILE_free_x(c) ((c)->cop_file = rcpv_free((c)->cop_file))
567-
# define CopFILE_copy_x(dst,src) ((dst)->cop_file = rcpv_copy((src)->cop_file))
569+
# define CopFILE_copy_x(dst,src) \
570+
((dst)->cop_file = rcpv_copy((src)->cop_file))
568571

569572
/* change condition to 1 && to enable this debugging */
570573
# define CopFILE_debug(c,t,rk) \
@@ -625,7 +628,8 @@ typedef struct rcpv RCPV;
625628
# define CopFILEGV_set(c,gv) ((c)->cop_filegv = (GV*)SvREFCNT_inc(gv))
626629
# define CopFILE_set(c,pv) CopFILEGV_set((c), gv_fetchfile(pv))
627630
# define CopFILE_copy(dst,src) CopFILEGV_set((dst),CopFILEGV(src))
628-
# define CopFILE_setn(c,pv,l) CopFILEGV_set((c), gv_fetchfile_flags((pv),(l),0))
631+
# define CopFILE_setn(c,pv,l) \
632+
CopFILEGV_set((c), gv_fetchfile_flags((pv),(l),0))
629633
# define CopFILESV(c) (CopFILEGV(c) ? GvSV(CopFILEGV(c)) : NULL)
630634
# define CopFILEAV(c) (CopFILEGV(c) ? GvAV(CopFILEGV(c)) : NULL)
631635
# ifdef DEBUGGING
@@ -642,7 +646,8 @@ typedef struct rcpv RCPV;
642646
? GvNAMELEN(CopFILEGV(c))-2 : 0)
643647
# define CopSTASH(c) ((c)->cop_stash)
644648
# define CopSTASH_set(c,hv) ((c)->cop_stash = (hv))
645-
# define CopFILE_free(c) (SvREFCNT_dec(CopFILEGV(c)),(CopFILEGV(c) = NULL))
649+
# define CopFILE_free(c) \
650+
(SvREFCNT_dec(CopFILEGV(c)),(CopFILEGV(c) = NULL))
646651

647652
#endif /* USE_ITHREADS */
648653

@@ -654,7 +659,8 @@ typedef struct rcpv RCPV;
654659
#define CopHINTHASH_get(c) ((COPHH*)((c)->cop_hints_hash))
655660
#define CopHINTHASH_set(c,h) ((c)->cop_hints_hash = (h))
656661

657-
#define CopFEATURES_setfrom(dst, src) ((dst)->cop_features = (src)->cop_features)
662+
#define CopFEATURES_setfrom(dst, src) \
663+
((dst)->cop_features = (src)->cop_features)
658664

659665
/*
660666
=for apidoc Am|SV *|cop_hints_fetch_pv |const COP *cop|const char *key |U32 hash|U32 flags
@@ -775,7 +781,8 @@ by setting C<*flags> to 0 or C<SVf_UTF8>.
775781

776782
#define CopLABEL(c) Perl_cop_fetch_label(aTHX_ (c), NULL, NULL)
777783
#define CopLABEL_len(c,len) Perl_cop_fetch_label(aTHX_ (c), len, NULL)
778-
#define CopLABEL_len_flags(c,len,flags) Perl_cop_fetch_label(aTHX_ (c), len, flags)
784+
#define CopLABEL_len_flags(c,len,flags) \
785+
Perl_cop_fetch_label(aTHX_ (c), len, flags)
779786
#define CopLABEL_alloc(pv) ((pv)?savepv(pv):NULL)
780787

781788
#define CopSTASH_ne(c,hv) (!CopSTASH_eq(c,hv))
@@ -933,7 +940,8 @@ struct block_loop {
933940

934941
#define CxLABEL(c) (CopLABEL((c)->blk_oldcop))
935942
#define CxLABEL_len(c,len) (CopLABEL_len((c)->blk_oldcop, len))
936-
#define CxLABEL_len_flags(c,len,flags) ((const char *)CopLABEL_len_flags((c)->blk_oldcop, len, flags))
943+
#define CxLABEL_len_flags(c,len,flags) \
944+
((const char *)CopLABEL_len_flags((c)->blk_oldcop, len, flags))
937945
#define CxHASARGS(c) (((c)->cx_type & CXp_HASARGS) == CXp_HASARGS)
938946

939947
/* CxLVAL(): the lval flags of the call site: the relevant flag bits from

cv.h

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -38,12 +38,14 @@ See L<perlguts/Autoloading with XSUBs>.
3838
# define Nullcv Null(CV*)
3939
#endif
4040

41-
#define CvSTASH(sv) (MUTABLE_HV(((XPVCV*)MUTABLE_PTR(SvANY(sv)))->xcv_stash))
41+
#define CvSTASH(sv) \
42+
(MUTABLE_HV(((XPVCV*)MUTABLE_PTR(SvANY(sv)))->xcv_stash))
4243
#define CvSTASH_set(cv,st) Perl_cvstash_set(aTHX_ cv, st)
4344
#define CvSTART(sv) ((XPVCV*)MUTABLE_PTR(SvANY(sv)))->xcv_start_u.xcv_start
4445
#define CvROOT(sv) ((XPVCV*)MUTABLE_PTR(SvANY(sv)))->xcv_root_u.xcv_root
4546
#define CvXSUB(sv) ((XPVCV*)MUTABLE_PTR(SvANY(sv)))->xcv_root_u.xcv_xsub
46-
#define CvXSUBANY(sv) ((XPVCV*)MUTABLE_PTR(SvANY(sv)))->xcv_start_u.xcv_xsubany
47+
#define CvXSUBANY(sv) \
48+
((XPVCV*)MUTABLE_PTR(SvANY(sv)))->xcv_start_u.xcv_xsubany
4749
#define CvGV(sv) Perl_CvGV(aTHX_ (CV *)(sv))
4850
#define CvGV_set(cv,gv) Perl_cvgv_set(aTHX_ cv, gv)
4951
#define CvHASGV(cv) cBOOL(SvANY(cv)->xcv_gv_u.xcv_gv)

dosish.h

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -21,7 +21,8 @@
2121
#else
2222
# define PERL_SYS_INIT_BODY(c,v) \
2323
MALLOC_CHECK_TAINT2(*c,*v); PERLIO_INIT
24-
# define BIT_BUCKET "\\dev\\nul" /* "wanna be like, umm, Newlined, or somethin?" */
24+
# define BIT_BUCKET "\\dev\\nul" /* "wanna be like, umm, Newlined,
25+
or somethin?" */
2526
#endif
2627

2728
/* Generally add things last-in first-terminated. IO and memory terminations

gv.h

Lines changed: 16 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -54,9 +54,12 @@ struct gp {
5454
assert(SvTYPE(_gvname_hek) == SVt_PVGV || SvTYPE(_gvname_hek) >= SVt_PVLV); \
5555
&(GvXPVGV(_gvname_hek)->xiv_u.xivu_namehek); \
5656
}))
57-
# define GvNAME_get(gv) ({ assert(GvNAME_HEK(gv)); (char *)HEK_KEY(GvNAME_HEK(gv)); })
58-
# define GvNAMELEN_get(gv) ({ assert(GvNAME_HEK(gv)); HEK_LEN(GvNAME_HEK(gv)); })
59-
# define GvNAMEUTF8(gv) ({ assert(GvNAME_HEK(gv)); HEK_UTF8(GvNAME_HEK(gv)); })
57+
# define GvNAME_get(gv) \
58+
({ assert(GvNAME_HEK(gv)); (char *)HEK_KEY(GvNAME_HEK(gv)); })
59+
# define GvNAMELEN_get(gv) \
60+
({ assert(GvNAME_HEK(gv)); HEK_LEN(GvNAME_HEK(gv)); })
61+
# define GvNAMEUTF8(gv) \
62+
({ assert(GvNAME_HEK(gv)); HEK_UTF8(GvNAME_HEK(gv)); })
6063
#else
6164
# define GvGP(gv) (0+(gv)->sv_u.svu_gp)
6265
# define GvGP_set(gv,gp) ((gv)->sv_u.svu_gp = (gp))
@@ -228,9 +231,11 @@ Return the CV from the GV.
228231
time, so don't try to free what's there. */
229232
#define GV_ADDMULTI 0x02 /* add, pretending it has been added
230233
already; used also by gv_init_* */
231-
#define GV_ADDWARN 0x04 /* add, but warn if symbol wasn't already there */
234+
#define GV_ADDWARN 0x04 /* add, but warn if symbol wasn't
235+
already there */
232236
/* 0x08 UNUSED */
233-
#define GV_NOINIT 0x10 /* add, but don't init symbol, if type != PVGV */
237+
#define GV_NOINIT 0x10 /* add, but don't init symbol,
238+
if type != PVGV */
234239
/* This is used by toke.c to avoid turing placeholder constants in the symbol
235240
table into full PVGVs with attached constant subroutines. */
236241
#define GV_NOADD_NOINIT 0x20 /* Don't add the symbol if it's not there.
@@ -273,9 +278,12 @@ Return the CV from the GV.
273278
#define gv_fetchsv_nomg(n,f,t) gv_fetchsv(n,(f)|GV_NO_SVGMAGIC,t)
274279
#define gv_init(gv,stash,name,len,multi) \
275280
gv_init_pvn(gv,stash,name,len,GV_ADDMULTI*cBOOL(multi))
276-
#define gv_fetchmeth(stash,name,len,level) gv_fetchmeth_pvn(stash, name, len, level, 0)
277-
#define gv_fetchmeth_autoload(stash,name,len,level) gv_fetchmeth_pvn_autoload(stash, name, len, level, 0)
278-
#define gv_fetchmethod_flags(stash,name,flags) gv_fetchmethod_pv_flags(stash, name, flags)
281+
#define gv_fetchmeth(stash,name,len,level) \
282+
gv_fetchmeth_pvn(stash, name, len, level, 0)
283+
#define gv_fetchmeth_autoload(stash,name,len,level) \
284+
gv_fetchmeth_pvn_autoload(stash, name, len, level, 0)
285+
#define gv_fetchmethod_flags(stash,name,flags) \
286+
gv_fetchmethod_pv_flags(stash, name, flags)
279287

280288
/*
281289
=for apidoc gv_autoload4

handy.h

Lines changed: 45 additions & 23 deletions
Original file line numberDiff line numberDiff line change
@@ -158,7 +158,8 @@ required, but is kept for backwards compatibility.
158158
* XXX Should really be a Configure probe, with HAS__FUNCTION__
159159
* and FUNCTION__ as results.
160160
* XXX Similarly, a Configure probe for __FILE__ and __LINE__ is needed. */
161-
#if (defined(__STDC_VERSION__) && __STDC_VERSION__ >= 199901L) || (defined(__SUNPRO_C)) /* C99 or close enough. */
161+
#if (defined(__STDC_VERSION__) && __STDC_VERSION__ >= 199901L) ||\
162+
(defined(__SUNPRO_C)) /* C99 or close enough. */
162163
# define FUNCTION__ __func__
163164
# define SAFE_FUNCTION__ __func__
164165
#elif (defined(__DECC_VER)) /* Tru64 or VMS, and strict C89 being used, but not modern enough cc (in Tru64, -c99 not known, only -std1). */
@@ -485,7 +486,8 @@ Perl_xxx(aTHX_ ...) form for any API calls where it's used.
485486
Perl_sv_catxmlpvn(aTHX_ dsv, STR_WITH_LEN(str), utf8)
486487

487488

488-
#define lex_stuff_pvs(pv,flags) Perl_lex_stuff_pvn(aTHX_ STR_WITH_LEN(pv), flags)
489+
#define lex_stuff_pvs(pv,flags) \
490+
Perl_lex_stuff_pvn(aTHX_ STR_WITH_LEN(pv), flags)
489491

490492
#define get_cvs(str, flags) \
491493
Perl_get_cvn_flags(aTHX_ STR_WITH_LEN(str), (flags))
@@ -712,7 +714,8 @@ based on the underlying C library functions):
712714
#define strnNE(s1,s2,l) (strncmp(s1,s2,l) != 0)
713715
#define strnEQ(s1,s2,l) (strncmp(s1,s2,l) == 0)
714716

715-
#define memEQ(s1,s2,l) (memcmp(((const void *) (s1)), ((const void *) (s2)), l) == 0)
717+
#define memEQ(s1,s2,l) \
718+
(memcmp(((const void *) (s1)), ((const void *) (s2)), l) == 0)
716719
#define memNE(s1,s2,l) (! memEQ(s1,s2,l))
717720

718721
/* memEQ and memNE where second comparand is a string constant */
@@ -744,7 +747,8 @@ based on the underlying C library functions):
744747
#define memGT(s1,s2,l) (memcmp(s1,s2,l) > 0)
745748
#define memGE(s1,s2,l) (memcmp(s1,s2,l) >= 0)
746749

747-
#define memCHRs(s1,c) ((const char *) memchr(ASSERT_IS_LITERAL(s1) , c, sizeof(s1)-1))
750+
#define memCHRs(s1,c) \
751+
((const char *) memchr(ASSERT_IS_LITERAL(s1) , c, sizeof(s1)-1))
748752

749753
/*
750754
* Character classes.
@@ -1666,7 +1670,8 @@ END_EXTERN_C
16661670
# define isPUNCT_A(c) generic_isCC_A_(c, CC_PUNCT_)
16671671
# define isSPACE_A(c) generic_isCC_A_(c, CC_SPACE_)
16681672
# define isWORDCHAR_A(c) generic_isCC_A_(c, CC_WORDCHAR_)
1669-
# define isXDIGIT_A(c) generic_isCC_(c, CC_XDIGIT_) /* No non-ASCII xdigits */
1673+
# define isXDIGIT_A(c) generic_isCC_(c, CC_XDIGIT_) /* No non-ASCII
1674+
xdigits */
16701675
# define isIDFIRST_A(c) generic_isCC_A_(c, CC_IDFIRST_)
16711676
# define isALPHA_L1(c) generic_isCC_(c, CC_ALPHA_)
16721677
# define isALPHANUMERIC_L1(c) generic_isCC_(c, CC_ALPHANUMERIC_)
@@ -2204,11 +2209,13 @@ END_EXTERN_C
22042209
#define isPRINT_uvchr(c) generic_invlist_uvchr_(CC_PRINT_, c)
22052210

22062211
#define isPUNCT_uvchr(c) generic_invlist_uvchr_(CC_PUNCT_, c)
2207-
#define isSPACE_uvchr(c) generic_uvchr_(CC_SPACE_, is_XPERLSPACE_cp_high, c)
2212+
#define isSPACE_uvchr(c) \
2213+
generic_uvchr_(CC_SPACE_, is_XPERLSPACE_cp_high, c)
22082214
#define isPSXSPC_uvchr(c) isSPACE_uvchr(c)
22092215

22102216
#define isUPPER_uvchr(c) generic_invlist_uvchr_(CC_UPPER_, c)
2211-
#define isVERTWS_uvchr(c) generic_uvchr_(CC_VERTSPACE_, is_VERTWS_cp_high, c)
2217+
#define isVERTWS_uvchr(c) \
2218+
generic_uvchr_(CC_VERTSPACE_, is_VERTWS_cp_high, c)
22122219
#define isWORDCHAR_uvchr(c) generic_invlist_uvchr_(CC_WORDCHAR_, c)
22132220
#define isXDIGIT_uvchr(c) generic_uvchr_(CC_XDIGIT_, is_XDIGIT_cp_high, c)
22142221

@@ -2749,7 +2756,8 @@ PoisonWith(0xEF) for catching access to freed memory.
27492756

27502757
#define MEM_SIZE_MAX ((MEM_SIZE)-1)
27512758

2752-
#define _PERL_STRLEN_ROUNDUP_UNCHECKED(n) (((n) - 1 + PERL_STRLEN_ROUNDUP_QUANTUM) & ~((MEM_SIZE)PERL_STRLEN_ROUNDUP_QUANTUM - 1))
2759+
#define _PERL_STRLEN_ROUNDUP_UNCHECKED(n) \
2760+
(((n) - 1 + PERL_STRLEN_ROUNDUP_QUANTUM) & ~((MEM_SIZE)PERL_STRLEN_ROUNDUP_QUANTUM - 1))
27532761

27542762
#ifdef PERL_MALLOC_WRAP
27552763

@@ -2803,7 +2811,8 @@ PoisonWith(0xEF) for catching access to freed memory.
28032811

28042812
# define MEM_WRAP_CHECK_(n,t) MEM_WRAP_CHECK(n,t),
28052813

2806-
# define PERL_STRLEN_ROUNDUP(n) ((void)(((n) > MEM_SIZE_MAX - 2 * PERL_STRLEN_ROUNDUP_QUANTUM) ? (croak_memory_wrap(),0) : 0), _PERL_STRLEN_ROUNDUP_UNCHECKED(n))
2814+
# define PERL_STRLEN_ROUNDUP(n) \
2815+
((void)(((n) > MEM_SIZE_MAX - 2 * PERL_STRLEN_ROUNDUP_QUANTUM) ? (croak_memory_wrap(),0) : 0), _PERL_STRLEN_ROUNDUP_UNCHECKED(n))
28072816
#else
28082817

28092818
# define MEM_WRAP_CHECK(n,t)
@@ -2863,9 +2872,12 @@ enum mem_log_type {
28632872
#endif
28642873

28652874
#ifdef PERL_MEM_LOG
2866-
#define MEM_LOG_ALLOC(n,t,a) Perl_mem_log_alloc(n,sizeof(t),STRINGIFY(t),a,__FILE__,__LINE__,FUNCTION__)
2867-
#define MEM_LOG_REALLOC(n,t,v,a) Perl_mem_log_realloc(n,sizeof(t),STRINGIFY(t),v,a,__FILE__,__LINE__,FUNCTION__)
2868-
#define MEM_LOG_FREE(a) Perl_mem_log_free(a,__FILE__,__LINE__,FUNCTION__)
2875+
#define MEM_LOG_ALLOC(n,t,a) \
2876+
Perl_mem_log_alloc(n,sizeof(t),STRINGIFY(t),a,__FILE__,__LINE__,FUNCTION__)
2877+
#define MEM_LOG_REALLOC(n,t,v,a) \
2878+
Perl_mem_log_realloc(n,sizeof(t),STRINGIFY(t),v,a,__FILE__,__LINE__,FUNCTION__)
2879+
#define MEM_LOG_FREE(a) \
2880+
Perl_mem_log_free(a,__FILE__,__LINE__,FUNCTION__)
28692881
#endif
28702882

28712883
#ifndef MEM_LOG_ALLOC
@@ -2878,9 +2890,12 @@ enum mem_log_type {
28782890
#define MEM_LOG_FREE(a) (a)
28792891
#endif
28802892

2881-
#define Newx(v,n,t) (v = (MEM_WRAP_CHECK_(n,t) (t*)MEM_LOG_ALLOC(n,t,safemalloc((MEM_SIZE)((n)*sizeof(t))))))
2882-
#define Newxc(v,n,t,c) (v = (MEM_WRAP_CHECK_(n,t) (c*)MEM_LOG_ALLOC(n,t,safemalloc((MEM_SIZE)((n)*sizeof(t))))))
2883-
#define Newxz(v,n,t) (v = (MEM_WRAP_CHECK_(n,t) (t*)MEM_LOG_ALLOC(n,t,safecalloc((n),sizeof(t)))))
2893+
#define Newx(v,n,t) \
2894+
(v = (MEM_WRAP_CHECK_(n,t) (t*)MEM_LOG_ALLOC(n,t,safemalloc((MEM_SIZE)((n)*sizeof(t))))))
2895+
#define Newxc(v,n,t,c) \
2896+
(v = (MEM_WRAP_CHECK_(n,t) (c*)MEM_LOG_ALLOC(n,t,safemalloc((MEM_SIZE)((n)*sizeof(t))))))
2897+
#define Newxz(v,n,t) \
2898+
(v = (MEM_WRAP_CHECK_(n,t) (t*)MEM_LOG_ALLOC(n,t,safecalloc((n),sizeof(t)))))
28842899

28852900
#ifndef PERL_CORE
28862901
/* pre 5.9.x compatibility */
@@ -2907,16 +2922,23 @@ enum mem_log_type {
29072922
#define perl_assert_ptr(p) assert( ((void*)(p)) != 0 )
29082923

29092924

2910-
#define Move(s,d,n,t) (MEM_WRAP_CHECK_(n,t) perl_assert_ptr(d), perl_assert_ptr(s), (void)memmove((char*)(d),(const char*)(s), (n) * sizeof(t)))
2911-
#define Copy(s,d,n,t) (MEM_WRAP_CHECK_(n,t) perl_assert_ptr(d), perl_assert_ptr(s), (void)memcpy((char*)(d),(const char*)(s), (n) * sizeof(t)))
2912-
#define Zero(d,n,t) (MEM_WRAP_CHECK_(n,t) perl_assert_ptr(d), (void)memzero((char*)(d), (n) * sizeof(t)))
2925+
#define Move(s,d,n,t) \
2926+
(MEM_WRAP_CHECK_(n,t) perl_assert_ptr(d), perl_assert_ptr(s), (void)memmove((char*)(d),(const char*)(s), (n) * sizeof(t)))
2927+
#define Copy(s,d,n,t) \
2928+
(MEM_WRAP_CHECK_(n,t) perl_assert_ptr(d), perl_assert_ptr(s), (void)memcpy((char*)(d),(const char*)(s), (n) * sizeof(t)))
2929+
#define Zero(d,n,t) \
2930+
(MEM_WRAP_CHECK_(n,t) perl_assert_ptr(d), (void)memzero((char*)(d), (n) * sizeof(t)))
29132931

29142932
/* Like above, but returns a pointer to 'd' */
2915-
#define MoveD(s,d,n,t) (MEM_WRAP_CHECK_(n,t) perl_assert_ptr(d), perl_assert_ptr(s), memmove((char*)(d),(const char*)(s), (n) * sizeof(t)))
2916-
#define CopyD(s,d,n,t) (MEM_WRAP_CHECK_(n,t) perl_assert_ptr(d), perl_assert_ptr(s), memcpy((char*)(d),(const char*)(s), (n) * sizeof(t)))
2917-
#define ZeroD(d,n,t) (MEM_WRAP_CHECK_(n,t) perl_assert_ptr(d), memzero((char*)(d), (n) * sizeof(t)))
2918-
2919-
#define PoisonWith(d,n,t,b) (MEM_WRAP_CHECK_(n,t) (void)memset((char*)(d), (U8)(b), (n) * sizeof(t)))
2933+
#define MoveD(s,d,n,t) \
2934+
(MEM_WRAP_CHECK_(n,t) perl_assert_ptr(d), perl_assert_ptr(s), memmove((char*)(d),(const char*)(s), (n) * sizeof(t)))
2935+
#define CopyD(s,d,n,t) \
2936+
(MEM_WRAP_CHECK_(n,t) perl_assert_ptr(d), perl_assert_ptr(s), memcpy((char*)(d),(const char*)(s), (n) * sizeof(t)))
2937+
#define ZeroD(d,n,t) \
2938+
(MEM_WRAP_CHECK_(n,t) perl_assert_ptr(d), memzero((char*)(d), (n) * sizeof(t)))
2939+
2940+
#define PoisonWith(d,n,t,b) \
2941+
(MEM_WRAP_CHECK_(n,t) (void)memset((char*)(d), (U8)(b), (n) * sizeof(t)))
29202942
#define PoisonNew(d,n,t) PoisonWith(d,n,t,0xAB)
29212943
#define PoisonFree(d,n,t) PoisonWith(d,n,t,0xEF)
29222944
#define Poison(d,n,t) PoisonFree(d,n,t)

hv.h

Lines changed: 10 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -25,11 +25,13 @@
2525
# define USE_PERL_PERTURB_KEYS 1
2626
# define PL_HASH_RAND_BITS_ENABLED PL_hash_rand_bits_enabled
2727
# endif
28-
# define PERL_HASH_ITER_BUCKET(iter) (((iter)->xhv_riter) ^ ((iter)->xhv_rand))
28+
# define PERL_HASH_ITER_BUCKET(iter) \
29+
(((iter)->xhv_riter) ^ ((iter)->xhv_rand))
2930
#endif
3031

3132
#ifdef PERL_USE_UNSHARED_KEYS_IN_LARGE_HASHES
32-
#define LARGE_HASH_HEURISTIC(hv,new_max) S_large_hash_heuristic(aTHX_ (hv), (new_max))
33+
#define LARGE_HASH_HEURISTIC(hv,new_max) \
34+
S_large_hash_heuristic(aTHX_ (hv), (new_max))
3335
#else
3436
#define LARGE_HASH_HEURISTIC(hv,new_max) 0
3537
#endif
@@ -366,8 +368,10 @@ whether it is valid to call C<HvAUX()>.
366368
#define HvUSEDKEYS(hv) (HvTOTALKEYS(hv) - HvPLACEHOLDERS_get(hv))
367369
#define HvTOTALKEYS(hv) (((XPVHV*) SvANY(hv))->xhv_keys)
368370
#define HvPLACEHOLDERS(hv) (*Perl_hv_placeholders_p(aTHX_ MUTABLE_HV(hv)))
369-
#define HvPLACEHOLDERS_get(hv) (SvMAGIC(hv) ? Perl_hv_placeholders_get(aTHX_ (const HV *)hv) : 0)
370-
#define HvPLACEHOLDERS_set(hv,p) Perl_hv_placeholders_set(aTHX_ MUTABLE_HV(hv), p)
371+
#define HvPLACEHOLDERS_get(hv) \
372+
(SvMAGIC(hv) ? Perl_hv_placeholders_get(aTHX_ (const HV *)hv) : 0)
373+
#define HvPLACEHOLDERS_set(hv,p) \
374+
Perl_hv_placeholders_set(aTHX_ MUTABLE_HV(hv), p)
371375

372376
/* This (now) flags whether *new* keys in the hash will be allocated from the
373377
* shared string table. We have a heuristic to call HvSHAREKEYS_off() if a hash
@@ -481,7 +485,8 @@ whether it is valid to call C<HvAUX()>.
481485
#define HV_ITERNEXT_WANTPLACEHOLDERS 0x01 /* Don't skip placeholders. */
482486

483487
#define hv_iternext(hv) hv_iternext_flags(hv, 0)
484-
#define hv_magic(hv, gv, how) sv_magic(MUTABLE_SV(hv), MUTABLE_SV(gv), how, NULL, 0)
488+
#define hv_magic(hv, gv, how) \
489+
sv_magic(MUTABLE_SV(hv), MUTABLE_SV(gv), how, NULL, 0)
485490
#define hv_undef(hv) Perl_hv_undef_flags(aTHX_ hv, 0)
486491

487492
#define Perl_sharepvn(pv, len, hash) HEK_KEY(share_hek(pv, len, hash))

0 commit comments

Comments
 (0)