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

Snowman, comet, regex parser, oh my! #1049

Closed
p6rt opened this issue Jun 11, 2009 · 5 comments
Closed

Snowman, comet, regex parser, oh my! #1049

p6rt opened this issue Jun 11, 2009 · 5 comments
Labels
Bug

Comments

@p6rt
Copy link

@p6rt p6rt commented Jun 11, 2009

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

Searchable as RT66498$

@p6rt
Copy link
Author

@p6rt p6rt commented Jun 11, 2009

From @masak

<masak> rakudo​: say "abc" ~~ m .(.).
<p6eval> rakudo 86aeaf​: OUTPUT«abc␤»
<masak> did somebody report that one?
<pmichaud> I don't know if it got rakudobugged.
<pmichaud> it did get mentioned during the design meeting
<pmichaud> I think my statement was
* masak rakudobugs
<pmichaud> "What in the world are you doing to the regex engine?!?"
<masak> :)
<masak> in a very spaced-out way, a snowman and a meteor are kinda symmetric.
<masak> but maybe not in the required way...
<pmichaud> rakudo​: say "abc" ~~ m .(.).; say 'ok';
<p6eval> rakudo 86aeaf​: OUTPUT«abc␤ok␤»
<pmichaud> rakudo​: say q.(.).;
<p6eval> rakudo 86aeaf​: OUTPUT«Statement not terminated properly [...]
<pmichaud> rakudo​: say "abc" ~~ m .(.).; say $0;
<p6eval> rakudo 86aeaf​: OUTPUT«abc␤b␤»
<pmichaud> looks like it must be something in the regex parser

@p6rt
Copy link
Author

@p6rt p6rt commented Nov 13, 2009

From @kyleha

This is an automatically generated mail to inform you that tests are now available in t/spec/S02-literals/quoting-unicode.t

commit 8cac88ed49a48592f4bdadfde9b49d19c4a6314d
Author​: kyle <kyle@​c213334d-75ef-0310-aa23-eaa082d1ae64>
Date​: Fri Nov 13 19​:05​:13 2009 +0000

  [t/spec] Test for RT 66498​: quoting regex with snowman and comet
 
  git-svn-id​: http://svn.pugscode.org/pugs@&#8203;29070 c213334d-75ef-0310-aa23-eaa082d1ae64

Inline Patch
diff --git a/t/spec/S02-literals/quoting-unicode.t b/t/spec/S02-literals/quoting-unicode.t
index 4d6ba4a..d7ba843 100644
--- a/t/spec/S02-literals/quoting-unicode.t
+++ b/t/spec/S02-literals/quoting-unicode.t
@@ -1,7 +1,6 @@
 use v6;
 use Test;
-
-plan 70;
+plan *;
 
 #L<S02/Literals>
 # TODO:
@@ -74,4 +73,15 @@ RIGHT SQUARE BRACKET WITH TICK IN BOTTOM CORNER(U+298D/U+298E)';
     is $var, 'b', 'q-style string with LEFT/RIGHT ANGLE BRACKET';
 }
 
+# RT #66498
+{
+    eval_dies_ok "q\c[SNOWMAN].\c[COMET]",
+        "Can't quote a string with a snowman and comet (U+2603 and U+2604)";
+    #?rakudo todo 'RT 66498'
+    eval_dies_ok "'RT 66498' ~~ m\c[SNOWMAN].\c[COMET]",
+        "Can't quote a regex with a snowman and comet (U+2603 and U+2604)";
+}
+
+done_testing;
+
 # vim: ft=perl6
@p6rt
Copy link
Author

@p6rt p6rt commented Nov 13, 2009

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

@p6rt
Copy link
Author

@p6rt p6rt commented Mar 20, 2010

From @masak

<masak> rakudo​: say "abc" ~~ m .(.).
<p6eval> rakudo 6c8737​: OUTPUT«Confused at line 11 [...]
<masak> \o/
<masak> for once, Rakudo and I are confused at the same thing! :)
* masak closes the infamous snowman/comet rakudobug :)
<masak> pmichaud++

@p6rt
Copy link
Author

@p6rt p6rt commented Mar 20, 2010

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

@p6rt p6rt closed this Mar 20, 2010
@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
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
1 participant
You can’t perform that action at this time.