Navigation Menu

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

ttNegative codepoint in .comb + /:m/ #5025

Closed
p6rt opened this issue Jan 9, 2016 · 6 comments
Closed

ttNegative codepoint in .comb + /:m/ #5025

p6rt opened this issue Jan 9, 2016 · 6 comments

Comments

@p6rt
Copy link

p6rt commented Jan 9, 2016

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

Searchable as RT127215$

@p6rt
Copy link
Author

p6rt commented Jan 9, 2016

From @Juerd

  $ perl6 -e'say "hello".comb(/​:m <[o]>/)'
  ===SORRY!===
  chr codepoint cannot be negative

--
Met vriendelijke groet, // Kind regards, // Korajn salutojn,

Juerd Waalboer <juerd@​tnx.nl>
TNX

@p6rt
Copy link
Author

p6rt commented Jan 9, 2016

From @lizmat

On 09 Jan 2016, at 06​:03, Juerd Waalboer (via RT) <perl6-bugs-followup@​perl.org> wrote​:

# New Ticket Created by Juerd Waalboer
# Please include the string​: [perl #​127215]
# in the subject line of all future correspondence about this issue.
# <URL​: https://rt-archive.perl.org/perl6/Ticket/Display.html?id=127215 >

$ perl6 -e'say "hello".comb(/​:m <[o]>/)'
===SORRY!===
chr codepoint cannot be negative

Please, note more information is available with —ll-exception. For bug reporting / triage, this can be very helpful.

--
Met vriendelijke groet, // Kind regards, // Korajn salutojn,

Juerd Waalboer <juerd@​tnx.nl>
TNX

Seems to happen somewhere deep in the NQP regex bowels.

$ 6l '"hello".comb(/​:m <[o]>/)'
chr codepoint cannot be negative
  at <unknown>​:1 (<ephemeral file>​::73)
from gen/moar/stage2/QRegex.nqp​:1337 (/Users/liz/Github/rakudo.moar/install/share/nqp/lib/QRegex.moarvm​:!cursor_more​:97)
from gen/moar/m-CORE.setting​:10144 (/Users/liz/Github/rakudo.moar/install/share/perl6/runtime/./CORE.setting.moarvm​::69)
from gen/moar/m-CORE.setting​:12487 (/Users/liz/Github/rakudo.moar/install/share/perl6/runtime/./CORE.setting.moarvm​::25)
from gen/moar/m-CORE.setting​:12519 (/Users/liz/Github/rakudo.moar/install/share/perl6/runtime/./CORE.setting.moarvm​:push-exactly​:98)

I haven’t been able to golf this further yet.

Liz

@p6rt
Copy link
Author

p6rt commented Jan 9, 2016

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

@p6rt
Copy link
Author

p6rt commented Sep 19, 2016

From @coke

On Fri Jan 08 21​:03​:49 2016, juerd@​tnx.nl wrote​:

$ perl6 \-e'say "hello"\.comb\(/&#8203;:m \<\[o\]>/\)'
===SORRY\!===
chr codepoint cannot be negative

--
Will "Coke" Coleda

I can no longer duplicate this​:

$ perl6 --version
This is Rakudo version 2016.08.1 built on MoarVM version 2016.08
implementing Perl 6.c.
$ perl6 -e'say "hello".comb(/​:m <[o]>/)'
(o)

Closable with tests.

@p6rt
Copy link
Author

p6rt commented Sep 20, 2016

From @zoffixznet

Tests added in Raku/roast@92e0197c3a

@p6rt
Copy link
Author

p6rt commented Sep 20, 2016

@zoffixznet - 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