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

No implicit <.ws> calls around the separator for <foo> ** \, in Rakudo (nqp-rx) #1805

Closed
p6rt opened this issue Jun 4, 2010 · 3 comments
Closed
Labels

Comments

@p6rt
Copy link

p6rt commented Jun 4, 2010

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

Searchable as RT75558$

@p6rt
Copy link
Author

p6rt commented Jun 4, 2010

From @masak

<moritz_> should rule { <value> ** \, } and rule { <value> ** [ \, ]
} be the same in terms of <.ws> handling?
<pmichaud> in ultimate effect they're largely the same
<pmichaud> there may be some case where they're not perfectly the same.
<pmichaud> because [ \, ] has some extra <.ws> calls
<moritz_> because in rakudo I see difference in the whitespace handling
<pmichaud> I'm not sure that rakudo/nqp gets the whitespace handling
around ** exactly right.
<pmichaud> PGE had it there, but I don't know that I ever restored it for nqp-rx
<pmichaud> I think nqp-rx still treats { <value> ** \, } exactly
the same as { <value>**\, }
<moritz_> that would explain my observations
<masak> ...which are...? :)
<moritz_> some of my tests fails when I write <value> ** \, but pass
if I write <value> ** [ \, ]
<moritz_> rakudo​: say 'a,b,c' ~~ /​:s <alpha> ** \,/
<p6eval> rakudo 2242ef​: OUTPUT«a,b,c␤»
<moritz_> rakudo​: say 'a, b, c' ~~ /​:s <alpha> ** \,/
<p6eval> rakudo 2242ef​: OUTPUT«a␤»
<masak> can I submit that to RT?
<moritz_> yes, please
* masak submits rakudobug

@p6rt
Copy link
Author

p6rt commented Jun 4, 2010

From @masak

<dalek> rakudo​: 8b116ff | moritz++ | build/PARROT_REVISION​:
<dalek> rakudo​: bump PARROT_REVISION to get latest nqp-rx fixes for ** quantifier and
:sigspace
<dalek> rakudo​: review​: http://github.com/rakudo/rakud[…]db725fcc3c258aa9a
<moritz_> phenny​: tell masak that his bug report for :sigpace + ** quantifier never made it to
RT, but it's fixed in 8b116ff in
<phenny> moritz_​: I'll pass that on when masak is around.

Resolving ticket.

@p6rt
Copy link
Author

p6rt commented Jun 4, 2010

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

@p6rt p6rt closed this as completed Jun 4, 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
Labels
Projects
None yet
Development

No branches or pull requests

1 participant