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
Rakudo fails to give "Null regex not allowed" for a number of operators #2155
Comments
From @masak<masak> std: "abc" ~~ /()/ This all comes down to how nqp-rx parses regexes. It can do it |
From @masak<moritz_> std: / / In the '|' case, it's OK for the left side to be empty, but there |
From @cokeOn Wed Mar 31 15:45:50 2010, masak wrote:
Error message has changed: 22:00 < [Coke]> rakudo: "abc" ~~ /()/ -- |
The RT System itself - Status changed from 'new' to 'open' |
From @diakopterOn Sun Feb 12 19:01:21 2012, coke wrote:
This has a couple later submitted duplicates. |
From @moritz< moritz> p: /a | / This should be reported as null pattern instead: < moritz> std: /a | / I have already added tests for that: commit 73208bff83283e4eb1b994bfcd5ef6fd885a00ec RT #78404: Error reporting on Null regexes Inline Patchdiff --git a/S32-exceptions/misc.t b/S32-exceptions/misc.t
index 0d4b9ed..d361f9c 100644
--- a/S32-exceptions/misc.t
+++ b/S32-exceptions/misc.t
@@ -133,6 +133,17 @@ throws_like 'my role R { has $.x; has $.y }; 99
+# RT #78404 throws_like 'sub f($a?, $b) { }', X::Parameter::WrongOrder, |
From @MouqThis has been fixed and the tests pass. Marking as resolved. |
The RT System itself - Status changed from 'new' to 'open' |
@Mouq - Status changed from 'open' to 'resolved' |
Migrated from rt.perl.org#77784 (status was 'resolved')
Searchable as RT77784$
The text was updated successfully, but these errors were encountered: