Skip Menu |

This queue is for tickets about the Test-Files CPAN distribution.

Report information
The Basics
Id: 133822
Status: open
Priority: 0/
Queue: Test-Files

People
Owner: Nobody in particular
Requestors: JSF [...] cpan.org
Cc: philcrow2000 [...] yahoo.com
AdminCc:

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



CC: philcrow2000 [...] yahoo.com
Subject: Caused crasg of Devel::Cover
Due to a bug in Devel::Cover (https://github.com/pjcj/Devel--Cover/issues/275) the following line in Test::Files causes a crash if I try to determine the test coverage of a script using Test::Files: if (index(ref $list, 'ARRAY') < 0) { This can easily be avoided without waiting of a Devel::Cover bug fixing (which can take a lot of time due to high complexity of this module) if this line would be replaced with the following one: if (ref($list) ne 'ARRAY') { If you are straitened for time to provide this change, I would love to get a co-maintainer and to provide this change by my own. Thanks and Regards, Jurij
Subject: Re: [rt.cpan.org #133822] Caused crasg of Devel::Cover
Date: Sat, 28 Nov 2020 06:19:27 -0600
To: bug-Test-Files [...] rt.cpan.org
From: Phil Crow <crow.phil [...] gmail.com>
I have added user JSF as a comaintainer of Test::Files. On Fri, Nov 27, 2020 at 9:40 AM Jurij Fajnberg via RT < bug-Test-Files@rt.cpan.org> wrote: Show quoted text
> Fri Nov 27 10:40:09 2020: Request 133822 was acted upon. > Transaction: Ticket created by JSF > Queue: Test-Files > Subject: Caused crasg of Devel::Cover > Broken in: (no value) > Severity: (no value) > Owner: Nobody > Requestors: JSF@cpan.org > Status: new > Ticket <URL: https://rt.cpan.org/Ticket/Display.html?id=133822 > > > > Due to a bug in Devel::Cover ( > https://github.com/pjcj/Devel--Cover/issues/275) the following line in > Test::Files causes a crash if I try to determine the test coverage of a > script using Test::Files: > > if (index(ref $list, 'ARRAY') < 0) { > > This can easily be avoided without waiting of a Devel::Cover bug fixing > (which can take a lot of time due to high complexity of this module) if > this line would be replaced with the following one: > > if (ref($list) ne 'ARRAY') { > > If you are straitened for time to provide this change, I would love to get > a co-maintainer and to provide this change by my own. > > Thanks and Regards, > Jurij >