| Subject: | Bleadperl ea25a9b breaks A:C 0.05 |
You may have noticed that recent bleadperl versions report a FAIL:
http://matrix.cpantesters.org/?dist=AnyEvent-Connection%200.05
The culprit is this commit as reported by git bisect:
commit ea25a9b2cf73948b1e8c5675de027e0ad13277bd
Author: Zefram <zefram@fysh.org>
Date: Wed Sep 8 09:51:29 2010
make qw(...) first-class syntax
This makes a qw(...) list literal a distinct token type for the
parser, where previously it was munged into a "(",THING,")" sequence.
The change means that qw(...) can't accidentally supply parens to parts
of the grammar that want real parens. Due to many bits of code taking
advantage of that by "foreach my $x qw(...) {}", this patch also
includes
a hack to coerce qw(...) to the old-style parenthesised THING, emitting
a deprecation warning along the way.
By virtue of the Test::NoWarnings your test uncovers this new warning
and consequently fails. The fix is trivial (tested). In
lib/AnyEvent/Connection.pm this pseudo patch:
BEGIN {
no strict 'refs';
- for my $m qw(push_write push_read unshift_read say reply recv
command want_command) {
+ for my $m (qw(push_write push_read unshift_read say reply recv
command want_command)) {
*$m = sub {
my $self = shift;
$self->{connected} or return $self->event( error
HTH, Thanks && Regards,