diff options
| author | Nicholas Clark <nick@ccl4.org> | 2008-10-27 20:25:46 +0000 |
|---|---|---|
| committer | Nicholas Clark <nick@ccl4.org> | 2008-10-27 20:25:46 +0000 |
| commit | b1bc3f345d4dfe19cd94c120c46649336b5cb92b (patch) | |
| tree | bdfcef0c7875037af6a5fa8195d2203fe76e6b27 /av.h | |
| parent | bb5dd93d07e2d54d352e519c7b43ff2d3e5c5f36 (diff) | |
| download | perl-b1bc3f345d4dfe19cd94c120c46649336b5cb92b.tar.gz | |
Add a macro MUTABLE_PTR(p), which on (non-pedantic) gcc will not cast
away const, returning a void *. Add MUTABLE_SV(sv) which uses this, and
replace all (SV *) casts either with MUTABLE_SV(sv), or (const SV *).
This probably still needs some work - assigning to SvPVX() and SvRV()
is now likely to generate a casting error. The core doesn't do this.
But as-is it's finding bugs that can be fixed.
p4raw-id: //depot/perl@34605
Diffstat (limited to 'av.h')
| -rw-r--r-- | av.h | 4 |
1 files changed, 2 insertions, 2 deletions
@@ -88,8 +88,8 @@ Same as C<av_len()>. Deprecated, use C<av_len()> instead. #define AvREALISH(av) (SvFLAGS(av) & (SVpav_REAL|SVpav_REIFY)) -#define AvFILL(av) ((SvRMAGICAL((SV *) (av))) \ - ? mg_size((SV *) av) : AvFILLp(av)) +#define AvFILL(av) ((SvRMAGICAL((const SV *) (av))) \ + ? mg_size(MUTABLE_SV(av)) : AvFILLp(av)) #define NEGATIVE_INDICES_VAR "NEGATIVE_INDICES" |