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

Rat-ifying a malformed number points to Metamodel.nqp (say “4​2”.Rat.nude) #6014

Closed
p6rt opened this issue Jan 23, 2017 · 4 comments
Closed
Labels
LTA Less Than Awesome; typically an error message that could be better regression Issue did not exist previously testneeded

Comments

@p6rt
Copy link

p6rt commented Jan 23, 2017

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

Searchable as RT130630$

@p6rt
Copy link
Author

p6rt commented Jan 23, 2017

From @AlexDaniel

Code​:
say �4�2�.Rat.nude

Result (2015.12,2016.06)​:
Cannot convert string to number​: trailing characters after number in '4â��â��2' (indicated by â��)
  in block <unit> at /tmp/WEcSEIS5b4 line 1

Actually thrown at​:
in block <unit> at /tmp/WEcSEIS5b4 line 1

Result (2016.12,HEAD)​:
Cannot convert string to number​: trailing characters after number in '4â��â��2' (indicated by â��)
  in block <unit> at /tmp/WEcSEIS5b4 line 1

Actually thrown at​:
  in any at gen/moar/Metamodel.nqp line 3072
in block <unit> at /tmp/WEcSEIS5b4 line 1

The error message should not contain lines from Metamodel.nqp.

Bisectable points to rakudo/rakudo@f294652

@p6rt
Copy link
Author

p6rt commented Jan 24, 2017

From @zoffixznet

Fixed with rakudo/rakudo@f230224d27 , tests needed

@p6rt
Copy link
Author

p6rt commented Mar 1, 2017

From @zoffixznet

Tests in rakudo/rakudo@cf935ca58f

@p6rt p6rt closed this as completed Mar 1, 2017
@p6rt
Copy link
Author

p6rt commented Mar 1, 2017

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

@p6rt p6rt added LTA Less Than Awesome; typically an error message that could be better regression Issue did not exist previously testneeded labels Jan 5, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
LTA Less Than Awesome; typically an error message that could be better regression Issue did not exist previously testneeded
Projects
None yet
Development

No branches or pull requests

1 participant