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

Rakudo stopped matching a regex, related to | vs. || (in 2015; better late than never) #2697

Open
raiph opened this issue Feb 14, 2019 · 1 comment

Comments

Projects
None yet
3 participants
@raiph
Copy link
Contributor

commented Feb 14, 2019

In the hope that it's OK to shift bugs from rt to here, this issue progresses one from 2015 in rt: https://rt.perl.org/Public/Bug/Display.html?id=125690 I haven't closed the rt bug yet. I can no longer login to rt. Can I close a bug via email? If so, how? TIA.

The Problem

Golfed from https://rt.perl.org/Public/Bug/Display.html?id=125690

say q[yz] ~~ token { w | [ x | y ] z } # 「yz」

say q[yz] ~~ token { w | [ x || y ] z } # Nil

Expected Behavior

From Moritz's comment in https://rt.perl.org/Public/Bug/Display.html?id=125690

I think this is a bug, because the two alternatives are mutually exclusive, and thus | vs || shouldn't matter.

The commit at fault

PerlPilot did the work to identify the commit that broke this in https://rt.perl.org/Public/Bug/Display.html?id=125690#txn-1367910

Hopefully my golf makes it easier to see if this is indeed a bug, Scott's work will make it pretty easy to spot what went wrong in nqp, and he's smiling down on me for my lame pun.

@AlexDaniel

This comment has been minimized.

Copy link
Member

commented Feb 14, 2019

I closed the ticket on RT for you.

@lucasbuchala lucasbuchala added the regex label Mar 14, 2019

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.