diff options
| author | Father Chrysostomos <sprout@cpan.org> | 2013-09-07 18:42:07 -0700 |
|---|---|---|
| committer | Father Chrysostomos <sprout@cpan.org> | 2013-09-07 18:42:07 -0700 |
| commit | 069420962bb857d2d053b8dd2bad601dd2a2d56e (patch) | |
| tree | c2ba71b0fb8284fa6915ac73b4afc3d5a74533e1 /proto.h | |
| parent | 851b527ad18524f953bb52604a2b669eb2e68648 (diff) | |
| download | perl-069420962bb857d2d053b8dd2bad601dd2a2d56e.tar.gz | |
Revert "Let av_push accept NULL values"
This reverts commit 7b6e8075e45ebc684565efbe3ce7b70435f20c79.
It turns out to be problematic, because it causes NULLs on the stack,
which XSUBs may trip on.
My main reason for it was actually to try to resolve some CPAN
failures, but it turns out that other fixes have removed the
need for that.
Diffstat (limited to 'proto.h')
| -rw-r--r-- | proto.h | 5 |
1 files changed, 3 insertions, 2 deletions
@@ -221,9 +221,10 @@ PERL_CALLCONV SV* Perl_av_pop(pTHX_ AV *av) assert(av) PERL_CALLCONV void Perl_av_push(pTHX_ AV *av, SV *val) - __attribute__nonnull__(pTHX_1); + __attribute__nonnull__(pTHX_1) + __attribute__nonnull__(pTHX_2); #define PERL_ARGS_ASSERT_AV_PUSH \ - assert(av) + assert(av); assert(val) PERL_CALLCONV void Perl_av_reify(pTHX_ AV *av) __attribute__nonnull__(pTHX_1); |