diff options
| author | David Mitchell <davem@iabyn.com> | 2015-11-19 10:03:04 +0000 |
|---|---|---|
| committer | David Mitchell <davem@iabyn.com> | 2015-11-19 10:30:54 +0000 |
| commit | 101d63659b7355370ff8d5a0076c0e456293458e (patch) | |
| tree | c792b195673b381c4d2af551dc75f9fd6a734dac /pp.h | |
| parent | 83144be336909f4c030bce1df7d2a06af2f25ec6 (diff) | |
| download | perl-101d63659b7355370ff8d5a0076c0e456293458e.tar.gz | |
silence compiler warnings using INCMARK/POPMARK
v5.23.3-305-g6cae08a introduced debugging variants of INCMARK/POPMARK, and
replaced a number of "PL_markstack_ptr--;" with "POPMARK;" etc.
This spews a bunch of "value computed is not used" warnings; so
add some "(void)"s.
Also indent the new definitions of INCMARK/POPMARK correctly.
Diffstat (limited to 'pp.h')
| -rw-r--r-- | pp.h | 18 |
1 files changed, 9 insertions, 9 deletions
@@ -56,7 +56,7 @@ Refetch the stack pointer. Used after a callback. See L<perlcall>. #define TARG targ #if defined(DEBUGGING) && defined(PERL_USE_GCC_BRACE_GROUPS) -#define PUSHMARK(p) \ +# define PUSHMARK(p) \ STMT_START { \ I32 * mark_stack_entry; \ if (UNLIKELY((mark_stack_entry = ++PL_markstack_ptr) == PL_markstack_max)) \ @@ -65,36 +65,36 @@ Refetch the stack pointer. Used after a callback. See L<perlcall>. DEBUG_s(PerlIO_printf(Perl_debug_log, "MARK push %p %d\n", \ PL_markstack_ptr, *mark_stack_entry)); \ } STMT_END -#define TOPMARK \ +# define TOPMARK \ ({ \ DEBUG_s(PerlIO_printf(Perl_debug_log, "MARK top %p %d\n", \ PL_markstack_ptr, *PL_markstack_ptr)); \ *PL_markstack_ptr; \ }) -#define POPMARK \ +# define POPMARK \ ({ \ DEBUG_s(PerlIO_printf(Perl_debug_log, "MARK pop %p %d\n", \ (PL_markstack_ptr-1), *(PL_markstack_ptr-1))); \ assert((PL_markstack_ptr > PL_markstack) || !"MARK underflow");\ *PL_markstack_ptr--; \ }) -#define INCMARK \ +# define INCMARK \ ({ \ DEBUG_s(PerlIO_printf(Perl_debug_log, "MARK inc %p %d\n", \ (PL_markstack_ptr+1), *(PL_markstack_ptr+1))); \ *PL_markstack_ptr++; \ }) #else -#define PUSHMARK(p) \ +# define PUSHMARK(p) \ STMT_START { \ I32 * mark_stack_entry; \ if (UNLIKELY((mark_stack_entry = ++PL_markstack_ptr) == PL_markstack_max)) \ mark_stack_entry = markstack_grow(); \ *mark_stack_entry = (I32)((p) - PL_stack_base); \ } STMT_END -#define TOPMARK (*PL_markstack_ptr) -#define POPMARK (*PL_markstack_ptr--) -#define INCMARK (*PL_markstack_ptr++) +# define TOPMARK (*PL_markstack_ptr) +# define POPMARK (*PL_markstack_ptr--) +# define INCMARK (*PL_markstack_ptr++) #endif #define dSP SV **sp = PL_stack_sp @@ -603,7 +603,7 @@ Does not use C<TARG>. See also C<L</XPUSHu>>, C<L</mPUSHu>> and C<L</PUSHu>>. while (jump_o->op_type == OP_NULL) \ jump_o = jump_o->op_next; \ assert(jump_o->op_type == OP_ENTERSUB); \ - POPMARK; \ + (void)POPMARK; \ return jump_o->op_next; \ } \ return NORMAL; \ |