develooper Front page | perl.perl5.porters | Postings from April 2018

Re: [perl #133142] Bleadperl breaks Devel::FindRef

Thread Previous | Thread Next
From:
Dave Mitchell
Date:
April 25, 2018 10:56
Subject:
Re: [perl #133142] Bleadperl breaks Devel::FindRef
Message ID:
20180425105619.GV597@iabyn.com
On Mon, Apr 23, 2018 at 04:52:26AM -0700, Sergey Aleynikov wrote:
> commit eacbb37937698a035d5ed63fcbdf15dd4eab56cf
> Author: Daniel Dragan <bulk88@hotmail.com>
> Date:   Fri Oct 31 03:23:17 2014 -0400
> 
>     free up CvPADLIST slot for XSUBs for future use
> 
> broke Devel::FindRef for perls compiled with -DDEBUGGING with the
> following assertion failure:
> 
> perl: FindRef.xs:180: XS_Devel__FindRef_find_: Assertion
> `!(((XPVCV*)({ void *_p = (((CV*)(sv))->sv_any); _p; }))->xcv_flags &
> 0x0008)' failed.
> 
> While a patch for Devel::FindRef is quite trivial, I'd like to know,
> are there any plans for CvPADLIST slots in XSUBs? Or isn't this
> assertion obsolete now?

I think Daniel would have to be the one to answer that question,
but in general, freeing up a slot in CV for potential future new uses 
seems like a Good Thing, and so the assertion isn't obsolete.

(Since this change was in 5.22.0, I'm removing this ticket from the 5.28
blockers list.)

-- 
In England there is a special word which means the last sunshine
of the summer. That word is "spring".

Thread Previous | Thread Next


nntp.perl.org: Perl Programming lists via nntp and http.
Comments to Ask Bjørn Hansen at ask@perl.org | Group listing | About