Skip Menu |
 

This queue is for tickets about the Devel-Size CPAN distribution.

Report information
The Basics
Id: 102909
Status: open
Priority: 0/
Queue: Devel-Size

People
Owner: Nobody in particular
Requestors: zefram [...] fysh.org
Cc:
AdminCc:

Bug Information
Severity: (no value)
Broken in: (no value)
Fixed in: (no value)



Subject: xsub padlist breakage in 5.21
Date: Fri, 20 Mar 2015 17:24:06 +0000
To: bug-Devel-Size [...] rt.cpan.org
From: Zefram <zefram [...] fysh.org>
Download (untitled) / with headers
text/plain 277b
Once methop is handled, in 5.21.6 Devel-Size crashes because it tries to look at the padlist of xsubs, which no longer have padlists and use the struct space for something else. Attached patch fixes. (But another problem is then revealed; separate ticket imminent.) -zefram
Download xsub_padlist.patch
text/x-diff 981b

Message body is not shown because sender requested not to inline it.

Download (untitled) / with headers
text/plain 744b
On Fri Mar 20 13:24:17 2015, zefram@fysh.org wrote: Show quoted text
> Once methop is handled, in 5.21.6 Devel-Size crashes because it tries to > look at the padlist of xsubs, which no longer have padlists and use the > struct space for something else. Attached patch fixes. (But another > problem is then revealed; separate ticket imminent.)
I already had a fix for this, which I hadn't uploaded as I was still attempting to figure out the Test::More::is() problem. This was the only one were I preferred my fix to yours, so I've not applied your patch. Sorry about the unintended duplication of work on this. However, your patch does act as code review on my fix, so it's not wasted effort. The bug is fixed in Devel-Size-0.79_52.tar.gz Nicholas Clark


This service is sponsored and maintained by Best Practical Solutions and runs on Perl.org infrastructure.

Please report any issues with rt.cpan.org to rt-cpan-admin@bestpractical.com.