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

Perl 5.20 regression: Hang when compiling while(1) in a list #14165

Closed
p5pRT opened this issue Oct 16, 2014 · 7 comments
Closed

Perl 5.20 regression: Hang when compiling while(1) in a list #14165

p5pRT opened this issue Oct 16, 2014 · 7 comments

Comments

@p5pRT
Copy link

p5pRT commented Oct 16, 2014

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

Searchable as RT122995$

@p5pRT
Copy link
Author

p5pRT commented Oct 16, 2014

From @cpansprout

This hangs at compile time​:

$ perl5.20.1 -e 'sub foo { () = ($a, my $b, ($c, do { while(1) {} })) }'
^C

I found this by reading the source code. (C.f. commit 71a29c3
which fixed a similar bug long ago.)

I’ll have a fix ready soon.

--

Father Chrysostomos

@p5pRT
Copy link
Author

p5pRT commented Oct 16, 2014

From @jkeenan

On Thu Oct 16 15​:53​:16 2014, sprout wrote​:

This hangs at compile time​:

$ perl5.20.1 -e 'sub foo { () = ($a, my $b, ($c, do { while(1) {} })) }'
^C

I found this by reading the source code. (C.f. commit 71a29c3
which fixed a similar bug long ago.)

I’ll have a fix ready soon.

Confirmed comparing 5.16.3 vs 5.20.1. Have you been able to bisect to determine the problematic commit?

Thank you very much.
Jim Keenan
--
James E Keenan (jkeenan@​cpan.org)

@p5pRT
Copy link
Author

p5pRT commented Oct 16, 2014

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

@p5pRT
Copy link
Author

p5pRT commented Oct 17, 2014

From @cpansprout

On Thu Oct 16 16​:33​:41 2014, jkeenan wrote​:

On Thu Oct 16 15​:53​:16 2014, sprout wrote​:

This hangs at compile time​:

$ perl5.20.1 -e 'sub foo { () = ($a, my $b, ($c, do { while(1) {}
})) }'
^C

I found this by reading the source code. (C.f. commit 71a29c3
which fixed a similar bug long ago.)

I’ll have a fix ready soon.

Confirmed comparing 5.16.3 vs 5.20.1. Have you been able to bisect to
determine the problematic commit?

It’s probably not necessary. The bad code was added by 7d3c8a6 according to git-blame.

I have just fixed it in 0d42e77, which should be backported to maint-5.20, IMO. (Please leave this ticket open till then.)

--

Father Chrysostomos

@p5pRT
Copy link
Author

p5pRT commented Dec 1, 2014

@steve-m-hay - Status changed from 'open' to 'pending release'

@p5pRT
Copy link
Author

p5pRT commented Jun 2, 2015

From @khwilliamson

Thanks for submitting this ticket

The issue should be resolved with the release today of Perl v5.22. If you find that the problem persists, feel free to reopen this ticket

--
Karl Williamson for the Perl 5 porters team

@p5pRT p5pRT closed this as completed Jun 2, 2015
@p5pRT
Copy link
Author

p5pRT commented Jun 2, 2015

@khwilliamson - Status changed from 'pending release' to 'resolved'

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