Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Bugs in extended regexp features #9750

Closed
p5pRT opened this issue May 28, 2009 · 9 comments
Closed

Bugs in extended regexp features #9750

p5pRT opened this issue May 28, 2009 · 9 comments

Comments

@p5pRT
Copy link

p5pRT commented May 28, 2009

Migrated from rt.perl.org#66104 (status was 'resolved')

Searchable as RT66104$

@p5pRT
Copy link
Author

p5pRT commented May 28, 2009

From @nwc10

Moritz Lenz mailed p5p in 477FACED.4000505@​casella.verplant.org
http​://www.xray.mpe.mpg.de/mailing-lists/perl5-porters/2008-01/msg00167.html

  With my shiny perl 5.10.0 on Linux I observed some bugs with extended
  regexp features. I know they're "experimental", but perhaps the reports
  help you anyway.

Dave notes​:

Bug #1 in the email still exists (bug 2 is fixed)​:

$ perl5.10.0 -Mre=eval -wlE 'my $re = qr/(\w+)/;"foo" =~ m/($re)(?{say +$^N})/'
Can't locate object method "say" via package "foo" (perhaps you forgot to load "foo"?) at (re_eval 1) line 1.

@p5pRT
Copy link
Author

p5pRT commented Jul 11, 2009

From @schwern

It appears no pragmas propagate into (?{ code }).

$ perl5.10.0 -Mre=eval -wlE 'my $re = qr/(\w+)/; use strict; "foo" =~
m/($re)(?{print +$bar})/'
Use of uninitialized value $bar in print at (re_eval 1) line 1.

$ perl5.10.0 -Mre=eval -wlE 'my $re = qr/(\w+)/; no warnings; "foo" =~
m/($re)(?{print +$bar})/'
Use of uninitialized value $bar in print at (re_eval 1) line 1.

This isn't new to 5.10.

$ perl5.8.8 -Mre=eval -wle 'my $re = qr/(\w+)/; no warnings; "foo" =~
m/($re)(?{print +$bar})/'
Use of uninitialized value in print at (re_eval 1) line 1.

Only the "use feature" pragma is new.

As its not a 5.10 regression, and (?{...}) is experimental, I'd suggest
removing it from the 5.10.1 blockers.

@p5pRT
Copy link
Author

p5pRT commented Jul 11, 2009

The RT System itself - Status changed from 'new' to 'open'

@p5pRT
Copy link
Author

p5pRT commented Jan 20, 2010

From @iabyn

On Sat, Jul 11, 2009 at 02​:02​:37AM -0700, Michael G Schwern via RT wrote​:

It appears no pragmas propagate into (?{ code }).

$ perl5.10.0 -Mre=eval -wlE 'my $re = qr/(\w+)/; use strict; "foo" =~
m/($re)(?{print +$bar})/'
Use of uninitialized value $bar in print at (re_eval 1) line 1.

$ perl5.10.0 -Mre=eval -wlE 'my $re = qr/(\w+)/; no warnings; "foo" =~
m/($re)(?{print +$bar})/'
Use of uninitialized value $bar in print at (re_eval 1) line 1.

This isn't new to 5.10.

$ perl5.8.8 -Mre=eval -wle 'my $re = qr/(\w+)/; no warnings; "foo" =~
m/($re)(?{print +$bar})/'
Use of uninitialized value in print at (re_eval 1) line 1.

Only the "use feature" pragma is new.

As its not a 5.10 regression, and (?{...}) is experimental, I'd suggest
removing it from the 5.10.1 blockers.

By the same logic, I don't think this bug needs to be a 5.12 showstopper.

Its also likely to be fixed as a side-effect my intended post-5.12
re-engineering of /(?{...})/ (which I've been promising for years....)

--
My Dad used to say 'always fight fire with fire', which is probably why
he got thrown out of the fire brigade.

@p5pRT
Copy link
Author

p5pRT commented Jan 20, 2010

From @obra

Only the "use feature" pragma is new.

As its not a 5.10 regression, and (?{...}) is experimental, I'd suggest
removing it from the 5.10.1 blockers.

By the same logic, I don't think this bug needs to be a 5.12 showstopper.

Its also likely to be fixed as a side-effect my intended post-5.12
re-engineering of /(?{...})/ (which I've been promising for years....)

I can buy that. I'm removing this as a blocker.

@p5pRT
Copy link
Author

p5pRT commented Dec 8, 2010

From @cpansprout

Fixed by 558b442.

@p5pRT
Copy link
Author

p5pRT commented Dec 8, 2010

@cpansprout - Status changed from 'open' to 'resolved'

@p5pRT p5pRT closed this as completed Dec 8, 2010
@p5pRT
Copy link
Author

p5pRT commented Dec 8, 2010

From @nwc10

On Tue Dec 07 18​:04​:16 2010, sprout wrote​:

Fixed by 558b442.

That revision doesn't exist. I guess you made a typo. Do you still have
the details of the revision handy?

Nicholas Clark

@p5pRT
Copy link
Author

p5pRT commented Dec 8, 2010

From @nwc10

On Wed Dec 08 01​:33​:44 2010, nicholas wrote​:

On Tue Dec 07 18​:04​:16 2010, sprout wrote​:

Fixed by 558b442.

That revision doesn't exist. I guess you made a typo. Do you still have
the details of the revision handy?

Aha. It does *now*. PEBKAC, at least when I thought I'd done a git pull.

Nicholas Clark

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant