Skip Menu |

This queue is for tickets about the Parallel-Depend CPAN distribution.

Report information
The Basics
Id: 48572
Status: resolved
Worked: 1 hour (60 min)
Priority: 0/
Queue: Parallel-Depend

People
Owner: LEMBARK [...] cpan.org
Requestors: SREZIC [...] cpan.org
Cc:
AdminCc:

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



Subject: Test failures with perl 5.10.1 (smart match-related?)
If you look at http://matrix.cpantesters.org/?dist=Parallel-Depend+4.09 then you see that Parallel-Depend is broken on perl 5.10.1. The error messages in the test reports suggest that it's related to smart matching (which changed in perl 5.10.1). Regards, Slaven
Subject: Re: [rt.cpan.org #48572] Test failures with perl 5.10.1 (smart match-related?)
Date: Mon, 10 Aug 2009 07:24:16 -0400
To: bug-Parallel-Depend [...] rt.cpan.org
From: Steven Lembark <lembark [...] wrkhors.com>
On Mon, 10 Aug 2009 02:15:40 -0400 "Slaven_Rezic via RT" <bug-Parallel-Depend@rt.cpan.org> wrote: Show quoted text
> Mon Aug 10 02:15:38 2009: Request 48572 was acted upon. > Transaction: Ticket created by SREZIC > Queue: Parallel-Depend > Subject: Test failures with perl 5.10.1 (smart match-related?) > Broken in: 4.09 > Severity: (no value) > Owner: Nobody > Requestors: SREZIC@cpan.org > Status: new > Ticket <URL: https://rt.cpan.org/Ticket/Display.html?id=48572 > > > > If you look at > http://matrix.cpantesters.org/?dist=Parallel-Depend+4.09 > then you see that Parallel-Depend is broken on perl 5.10.1. The error > messages in the test reports suggest that it's related to smart > matching (which changed in perl 5.10.1).
Thanks, I've already seen it. If you look at the error message, it is complaining about a smart match on an blessed refrences. You have to overload any smart matches on blessed ref's, which is a huge overhead for anything used frequently in the code. That or you have to de-reference it (e.g., $a ~~ %{ $hashref }) everywere in the base class' code. Change the structure internally and you get to hack the source code with a new sigil. This was intended to improve encapsulation. I've removed all of the smart matches from all of my code -- they are now officially a waste of time. The next chance I get to upload a version it'll fix 5.10.1 (and 5.11 also). enjoi -- Steven Lembark 85-09 90th St. Workhorse Computing Woodhaven, NY, 11421 lembark@wrkhors.com +1 888 359 3508
Should be fixed in 4.10 (on CPAN now). I've removed all smart matches from the code -- overloading or de- referencing the contents were too expensive (at least in 5.10.0 that I have here).
bad click, shouldn't have re-opened it.
On Tue Aug 11 11:33:32 2009, LEMBARK wrote: Still the matrix shows everything in red for perl 5.10.1: http://matrix.cpantesters.org/?dist=Parallel-Depend%204.10 Regards, Slaven
On Tue Nov 03 19:01:21 2009, SREZIC wrote: Show quoted text
> On Tue Aug 11 11:33:32 2009, LEMBARK wrote: > > Still the matrix shows everything in red for perl 5.10.1: > http://matrix.cpantesters.org/?dist=Parallel-Depend%204.10 >
Ah sorry, I did not see that there was also a 4.10.1... Regards, Slaven
Subject: Re: [rt.cpan.org #48572] Test failures with perl 5.10.1 (smart match-related?)
Date: Tue, 3 Nov 2009 19:25:50 -0500
To: bug-Parallel-Depend [...] rt.cpan.org
From: Steven Lembark <lembark [...] wrkhors.com>
On Tue, 03 Nov 2009 19:02:09 -0500 "Slaven_Rezic via RT" <bug-Parallel-Depend@rt.cpan.org> wrote: Show quoted text
> Ah sorry, I did not see that there was also a 4.10.1...
If that's the worst thing that happens to me today I'll be fine :-) enjoi -- Steven Lembark 85-09 90th St. Workhorse Computing Woodhaven, NY, 11421 lembark@wrkhors.com +1 888 359 3508