Skip Menu |

Preferred bug tracker

Please visit the preferred bug tracker to report your issue.

This queue is for tickets about the Inline CPAN distribution.

Report information
The Basics
Id: 6327
Status: resolved
Priority: 0/
Queue: Inline

People
Owner: Nobody in particular
Requestors: at [...] altlinux.org
Cc:
AdminCc:

Bug Information
Severity: Important
Broken in: (no value)
Fixed in: 0.44



Subject: Inline::C::ParseRegExp doesnot work with perl-5.8.4
Inline::C::ParseRegExp doesnot pass syntax check because of recent changes in perl-5.8.4. This was reported to p5p and inline mailing lists. Please see for details: http://www.nntp.perl.org/group/perl.inline/2273 http://www.nntp.perl.org/group/perl.inline/2274 -- Alexey Tourbin ALT Linux Team
--- Inline-0.44/C/lib/Inline/C/ParseRegExp.pm- 2002-11-04 21:39:52 +0000 +++ Inline-0.44/C/lib/Inline/C/ParseRegExp.pm 2004-05-15 17:14:19 +0000 @@ -21,9 +21,9 @@ sub code { my $RE_comment_Cpp = q{(?:\/\*(?:(?!\*\/)[\s\S])*\*\/|\/\/[^\n]*\n)}; my $RE_quoted = (q{(?:(?:\")(?:[^\\\"]*(?:\\.[^\\\"]*)*)(?:\")} .q{|(?:\')(?:[^\\\']*(?:\\.[^\\\']*)*)(?:\'))}); - our $RE_balanced_brackets = + our $RE_balanced_brackets; $RE_balanced_brackets = qr'(?:[{]((?:(?>[^{}]+)|(??{$RE_balanced_brackets}))*)[}])'; - our $RE_balanced_parens = + our $RE_balanced_parens; $RE_balanced_parens = qr'(?:[(]((?:(?>[^()]+)|(??{$RE_balanced_parens}))*)[)])'; # First, we crush out anything potentially confusing.
From: SISYPHUS [...] cpan.org
On Mon May 17 09:19:18 2004, guest wrote: Show quoted text
> Inline::C::ParseRegExp doesnot pass syntax check because of recent > changes in perl-5.8.4. This was reported to p5p and inline mailing > lists. Please see for details: > http://www.nntp.perl.org/group/perl.inline/2273 > http://www.nntp.perl.org/group/perl.inline/2274
In addition, I've found it necessary to comment out the following line of code in ParseRegExp.pm's code() subroutine: $code =~ s/$RE_quoted/\"\"/go; It's possible that commenting out that line breaks something - if it does, then I've not yet been bitten by that breakage (though I have been bitten by allowing that line of code to remain). Inline::C::ParseRegExp.pm is so much faster than Parse::RecDescent.pm that it's well worth using if there's a lot of C code. It's a pity that Inline::C::ParseRegExp.pm is not being maintained. The author, Mitch Charity, is apparently, along with Neil Watkiss, yet another graduate of the Brian Ingerson School of Total Committment. Cheers, Rob
RT-Send-CC: sisyphus [...] cpan.org
This has, I believe, been addressed in Inline-0.45 and a test added to the test suite (C/t/06parseregexp.t). If there's a problem with the way this has been dealt with, please feel free to re-open the ticket (or submit a new report). To further discuss the issue (but without re-opening the ticket), consider posting to the Inline mailing list, or send a private email to sisyphus at cpan dot org. Thanks for the report !! Cheers, Rob