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

Meta-error when trying to find 'panic' in a faulty regex in Rakudo #1716

Closed
p6rt opened this issue May 1, 2010 · 5 comments
Closed

Meta-error when trying to find 'panic' in a faulty regex in Rakudo #1716

p6rt opened this issue May 1, 2010 · 5 comments
Labels

Comments

@p6rt
Copy link

p6rt commented May 1, 2010

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

Searchable as RT74832$

@p6rt
Copy link
Author

p6rt commented May 1, 2010

From @masak

<moritz_> rakudo​: / a+ +/
<p6eval> rakudo c4857a​: OUTPUT«Method 'panic' not found for invocant
of class 'Regex;Match' [...]
<moritz_> masak​: care to submit?
* masak submits rakudobug
<moritz_> error while trying to produce an error message, it seems
<masak> how meta.

@p6rt
Copy link
Author

p6rt commented May 29, 2010

From @jnthn

On Sat May 01 13​:40​:04 2010, masak wrote​:

<moritz_> rakudo​: / a+ +/
<p6eval> rakudo c4857a​: OUTPUT«Method 'panic' not found for invocant
of class 'Regex;Match' [...]
<moritz_> masak​: care to submit?
* masak submits rakudobug
<moritz_> error while trying to produce an error message, it seems
<masak> how meta.

Fixed it; now we get the expected error​:

/ a+ +/
Can't quantify zero-width atom at line 1

Given to moritz++ for spectests.

Thanks,

Jonathan

@p6rt
Copy link
Author

p6rt commented May 29, 2010

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

@p6rt
Copy link
Author

p6rt commented May 30, 2010

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

@p6rt p6rt closed this as completed May 30, 2010
@p6rt
Copy link
Author

p6rt commented May 30, 2010

From @kyleha

This is an automatically generated mail to inform you that tests are now available in t/spec/S05-metasyntax/unknown.t

commit cb65adf5177e781c203e37e0ba4a9df6b372ccc8
Author​: moritz <moritz@​c213334d-75ef-0310-aa23-eaa082d1ae64>
Date​: Sun May 30 13​:54​:24 2010 +0000

  [t/spec] tests for RT #​74832
 
  git-svn-id​: http://svn.pugscode.org/pugs@&#8203;30995 c213334d-75ef-0310-aa23-eaa082d1ae64

Inline Patch
diff --git a/t/spec/S05-metasyntax/unknown.t b/t/spec/S05-metasyntax/unknown.t
index 0ec2ffb..8b18392 100644
--- a/t/spec/S05-metasyntax/unknown.t
+++ b/t/spec/S05-metasyntax/unknown.t
@@ -1,7 +1,7 @@
 use v6;
 use Test;
 
-plan 8;
+plan 10;
 
 # L<S05/Simplified lexical parsing of patterns/not all non-identifier glyphs are currently meaningful as metasyntax>
 
@@ -24,4 +24,10 @@ lives_ok({"aa!" ~~ /'a'/}, 'quoted "a" is valid');
     ok '!' !~~ /<foo>/, '\\{ in a rule (-)';
 }
 
+# RT #74832
+{
+    nok eval('/ a+ + /'), 'Cannot parse regex a+ +';
+    ok "$!" ~~ /quantif/, 'error message mentions quantif{y,ier}';
+}
+
 # vim: ft=perl6

@p6rt p6rt added the Bug label Jan 5, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

1 participant