Branch: refs/heads/blead
Home: https://github.com/Perl/perl5
Commit: 811f8a246ee7da5274c3229345222e9fe5febbe0
https://github.com/Perl/perl5/commit/811f8a246ee7da5274c3229345222e9fe5febbe0
Author: Karl Williamson <[email protected]>
Date: 2020-01-02 (Thu, 02 Jan 2020)
Changed paths:
M av.c
Log Message:
-----------
Revert "Add some defensive coding to av_store()"
This reverts commit bc62bf8519f9005df2fb29dbd3d330202b258b6b.
As Dave Mitchell said in <Perl/perl5/issues/17265/[email protected]>
"The docs for av_store() say that the caller is responsible for first
incrementing the ref count of the new SV before passing it to
av_store(). In the normal case of the two elements being different,
av_store() will decrement the old SV before storing the new one, and
everything is good. When the two elements are the same SV, av_store()
*still* needs to decrement the ref count, to undo the increment the
caller just did."
This should resolve GH #17265
