Skip Menu |

This queue is for tickets about the Want CPAN distribution.

Report information
The Basics
Id: 100626
Status: resolved
Priority: 0/
Queue: Want
People
Owner: robin [...] cpan.org
Requestors: 'spro^^*%*^6ut# [...] &$%*c
Cc:
AdminCc:
Bug Information
Severity: (no value)
Broken in: (no value)
Fixed in: (no value)

History Show all quoted text
  Mon Dec 01 01:06:38 2014 $_ = 'spro^^*%*^6ut# [...] &$%*c>#!^!#&!pan.org'; y/a-z.@//cd; print - Ticket created
Subject: [PATCH] Adjust tests for 5.21.7
I think the previous behaviour of making \scalar(xxx) put xxx in rvalue context was wrong, because \scalar($x) returns a reference to $x, but \scalar(pos) doesn’t return a reference to something that, when assigned to, sets pos. So I fixed this in perl commit 569ddb4. But Want was testing for the previous behaviour.
Subject: open_LdFf9m4w.txt
--- a/t/all.t 2014-11-30 22:02:41.000000000 -0800 +++ b/t/all.t 2014-11-30 22:02:31.000000000 -0800 @@ -140,7 +140,7 @@ print ($y == 23 ? "ok 35\n" : "not ok 35 @x = g(36, 'RVALUE', 'LIST'); @x = \(g(37, 'LVALUE', 'LIST')); -($x) = \(scalar g(38, 'RVALUE')); +($x) = \(scalar g(38, $] >= 5.021007 ? ('LVALUE', 'SCALAR') : 'RVALUE')); $$x = 29; # There used to be a test here which tested that $y != 29. However this
  Tue Dec 02 07:19:20 2014 robin [...] cpan.org - Correspondence added
On Mon Dec 01 01:06:38 2014, SPROUT wrote: Show quoted text
> I think the previous behaviour of making \scalar(xxx) put xxx in > rvalue context was wrong, because \scalar($x) returns a reference to > $x, but \scalar(pos) doesn’t return a reference to something that, > when assigned to, sets pos. So I fixed this in perl commit 569ddb4. > But Want was testing for the previous behaviour.
Thanks again! Applied and uploaded as 0.24.
  Tue Dec 02 07:19:21 2014 The RT System itself - Status changed from 'new' to 'open'
  Tue Dec 02 07:19:24 2014 robin [...] cpan.org - Status changed from 'open' to 'resolved'
  Tue Dec 02 07:19:24 2014 robin [...] cpan.org - Taken