Bug #73666 for String-Defer: Bleadperl v5.15.5-286-g83f78d1 breaks BMORROW/String-Defer-2.tar.gz

Mon Jan 02 02:29:40 2012 ANDK [...] cpan.org - Ticket created

Subject:

Bleadperl v5.15.5-286-g83f78d1 breaks BMORROW/String-Defer-2.tar.gz

Cross reference: https://rt.perl.org:443/rt3/Ticket/Display.html?id=107388 HTH

Mon Jan 02 20:57:32 2012 $_ = 'spro^^*%*^6ut# [...] &$%*c>#!^!#&!pan.org'; y/a-z.@//cd; print - Correspondence added

On Mon Jan 02 02:29:40 2012, ANDK wrote: Show quoted text

> Cross reference: > > https://rt.perl.org:443/rt3/Ticket/Display.html?id=107388

Here is a patch to work around the perl change. (BTW, I consider this a perl bug fix.)

Subject:

open_mOJnXARm.txt

diff -rup String-Defer-2-yLF2Ll/t/ref.t String-Defer-2-yLF2Ll-copy/t/ref.t --- String-Defer-2-yLF2Ll/t/ref.t 2011-11-01 15:01:31.000000000 -0700 +++ String-Defer-2-yLF2Ll-copy/t/ref.t 2012-01-02 17:56:08.000000000 -0800 @@ -50,8 +50,14 @@ sub check { check \substr($targ, 1, 3), sub { $targ = "XfooX"; "foo" }, "\\substr()"; + my $expect = do { + my $targ2 = "X"; + my $substr = \substr $targ2, 1, 3; + $targ2 = "XXXXX"; + $$substr + }; $targ = "X"; - check \substr($targ, 1, 3), sub { $targ = "XXXXX"; "" }, + check \substr($targ, 1, 3), sub { $targ = "XXXXX"; $expect }, "\\substr(<outside>)"; }

Mon Jan 02 20:57:34 2012 The RT System itself - Status changed from 'new' to 'open'

Mon Jan 02 21:46:25 2012 ben [...] morrow.me.uk - Correspondence added

Subject:	Re: [rt.cpan.org #73666] Bleadperl v5.15.5-286-g83f78d1 breaks BMORROW/String-Defer-2.tar.gz
Date:	Tue, 3 Jan 2012 02:46:09 +0000
To:	Father Chrysostomos via RT <bug-String-Defer [...] rt.cpan.org>
From:	Ben Morrow <ben [...] morrow.me.uk>

At 8PM -0500 on 2/01/12 you (Father Chrysostomos via RT) wrote: Show quoted text

> > Here is a patch to work around the perl change.

Thank you. I'll upload a fixed version when I get to it. Show quoted text

> (BTW, I consider this a perl bug fix.)

Me too. Ben

Tue Jan 03 15:25:21 2012 BMORROW [...] cpan.org - Correspondence added

I've just uploaded v3, which resolves this.

Tue Jan 03 15:25:22 2012 BMORROW [...] cpan.org - Status changed from 'open' to 'resolved'