Bug #32010 for HTML-TableContentParser: Test fails under Perl 5.10.0 under Mac OS X 10.3.9

Sun Dec 30 13:11:02 2007 wyant [...] cpan.org - Ticket created

Subject:

Test fails under Perl 5.10.0 under Mac OS X 10.3.9

What appears to be happening is that $#{@$tables} is coming up -1 under Perl 5.10.0, so the header and content tests get skipped. The perldelta for 5.10 says that the $# variable has been removed, but that's the default output format for numbers. Maybe someone went overboard? I have reported this to the Perl porters; it's bug 49230; see http://rt.perl.org/rt3/Public/Bug/Display.html?id=49230 I have attached 1.pat, a universal diff showing one way to deal with this. Thank you very much for your time and trouble.

Subject:

1.pat

Download 1.pat
application/octet-stream 588b

Message body not shown because it is not plain text.

Sun Dec 30 21:13:56 2007 wyant [...] cpan.org - Correspondence added

The Perl5 porters tell me that the change in behavior of $#{@$x} is a bug fix, which would be obvious with 'use strict;' asserted. The deal is that the block inside the brackets is evaluated in scalar context, so @$x becomes, literally, 3.

Sun Dec 30 21:14:00 2007 wyant [...] cpan.org - Status changed from 'new' to 'open'

Mon Dec 31 13:23:18 2007 wyant [...] cpan.org - Correspondence added

The follow-up to the Perl bug report has finally provided the supported way to do $#{@$x}. It's $#$x, which I guess makes sense if you think about it a bit. Anyhow, please ignore my original patch, since this looks like a better way to do it.

Sat Apr 15 12:18:59 2017 wyant [...] cpan.org - Correspondence added

I have forked gitpan/HTML-TableContentParser as trwyant/perl-HTML-TableContentParser, and applied a patch for this and 80317 Undeclared dependency on HTML::Parser. If you are willing to grant me co-maintainer status I am willing to make a release.

Wed Dec 27 14:57:19 2017 wyant [...] cpan.org - Status changed from 'open' to 'patched'

Wed Dec 27 14:57:19 2017 wyant [...] cpan.org - Fixed in 0.200 added

Wed Jan 03 10:57:43 2018 wyant [...] cpan.org - Status changed from 'patched' to 'resolved'