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
[BUG all] 'die qr{pattern}' does not check termination #2931
Comments
From @vanstynWith 5.6.0 for example: Hugo Summary of my perl5 (revision 5.0 version 6 subversion 0) configuration: Characteristics of this binary (from libperl): |
From @simoncozensOn Sun, Dec 03, 2000 at 05:16:40AM +0000, Hugo wrote:
Inline Patch--- ../utfperl/pp_sys.c Sat Dec 9 17:19:05 2000
+++ pp_sys.c Sun Dec 10 00:54:32 2000
@@ -448,7 +448,7 @@
}
else {
tmpsv = TOPs;
- tmps = SvROK(tmpsv) ? Nullch : SvPV(tmpsv, len);
+ tmps = (SvROK(tmpsv) && PL_in_eval) ? Nullch : SvPV(tmpsv, len);
}
if (!tmps || !len) {
SV *error = ERRSV; |
From @nwc10On Sat Dec 02 12:13:26 2000, hv@crypt.compulink.co.uk wrote:
With 5.8.1 for example $ ~/Sandpit/581/bin/perl -e 'die qr{x}' contrast with 5.8.0 $ ~/Sandpit/580/bin/perl -e 'die qr{x}' so this certainly isn't *resolved*, which was its status in RT. The fix for this bug was reversed in commit bf12611, due See this comment from Hugo in : I'm not sure how we can improve on this. The best suggestion I can come It's not clear how to resolve both #17763 and #4821 at the same time. Nicholas Clark |
@nwc10 - Status changed from 'resolved' to 'open' |
From @dcollinsnThis is broken in blead, marking stalled. TODO test will be forthcoming. |
From [Unknown Contact. See original ticket]This is broken in blead, marking stalled. TODO test will be forthcoming. |
@dcollinsn - Status changed from 'open' to 'stalled' |
Migrated from rt.perl.org#4821 (status was 'stalled')
Searchable as RT4821$
The text was updated successfully, but these errors were encountered: