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 allows unspace in /\ X/ but STD doesn't #2107

Closed
p6rt opened this issue Aug 30, 2010 · 8 comments
Closed

Rakudo allows unspace in /\ X/ but STD doesn't #2107

p6rt opened this issue Aug 30, 2010 · 8 comments

Comments

@p6rt
Copy link

p6rt commented Aug 30, 2010

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

Searchable as RT77522$

@p6rt
Copy link
Author

p6rt commented Aug 30, 2010

From @bbkr

[18​:51] <bbkr> rakudo​: /\ X/ # unspace bug :) known?
[18​:51] <p6eval> rakudo 6ccdcd​: ( no output )

[18​:51] <bbkr> std​: /\ X/
[18​:51] <p6eval> std 32116​: OUTPUT«Invalid conversion in sprintf​: "%M" at
STD.pm line 57284.��[31m===�[0mSORRY!�[31m===�[0m�No unspace allowed in
regex; if you meant to match the literal character, please enclose in single
quotes (' ') or use a backslashed form like \x%M->{02}x at /tmp/p05dWQrWFG
line 1​:â�¤------>
[18​:51] <p6eval> ..�[3â�¦

@p6rt
Copy link
Author

p6rt commented Jul 2, 2012

From @bbkr

17​:49 bbkr​: rakudo​: /\ X/
17​:49 p6eval​: rakudo 16db64​: ( no output )

17​:49 bbkr​: rakudo​: say " X" ~~ /\ X/
17​:49 p6eval​: rakudo 16db64​: OUTPUT«q[ X]â�¤â�¤Â»

Marking as testneeded.

@p6rt
Copy link
Author

p6rt commented Jul 2, 2012

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

@p6rt
Copy link
Author

p6rt commented Jul 15, 2012

From @bbkr

My mistake, rakudo should NOT match, STD disallows unspaces.

Removing testneeded tag, this is still broken.

@p6rt
Copy link
Author

p6rt commented Apr 13, 2014

From @Mouq

On Sun Jul 15 03​:49​:09 2012, bbkr wrote​:

My mistake, rakudo should NOT match, STD disallows unspaces.

Removing testneeded tag, this is still broken.

Dies correctly now​:

$ perl6 -e'/\ X/'
===SORRY!=== Error while compiling -e
No unspace allowed in regex; if you meant to match the literal character, please enclose in single quotes (' ') or use a backslashed form like \x20
at -e​:1
------> /\ �X/

Just needs tests

@p6rt
Copy link
Author

p6rt commented Aug 11, 2014

From @peschwa

Am Sa 12. Apr 2014, 19​:49​:11, Mouq schrieb​:

On Sun Jul 15 03​:49​:09 2012, bbkr wrote​:

My mistake, rakudo should NOT match, STD disallows unspaces.

Removing testneeded tag, this is still broken.

Dies correctly now​:

$ perl6 -e'/\ X/'
===SORRY!=== Error while compiling -e
No unspace allowed in regex; if you meant to match the literal
character, please enclose in single quotes (' ') or use a backslashed
form like \x20
at -e​:1
------> /\ �X/

Just needs tests

A typed exception has been added to Rakudo (X​::Syntax​::Regex​::Unspace) and a test as per above has been added to roast in commit c1f9b00f.

@p6rt
Copy link
Author

p6rt commented Sep 24, 2014

From @usev6

The test in S32-exceptions/misc.t passes and the test file is in 't/spectest.data'.

If the ticket should not be set to resolved (if there is something else to do), please comment.

@p6rt
Copy link
Author

p6rt commented Sep 24, 2014

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant