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

LTA message on missing radix conversion param #2745

Closed
p6rt opened this issue May 2, 2012 · 5 comments
Closed

LTA message on missing radix conversion param #2745

p6rt opened this issue May 2, 2012 · 5 comments

Comments

@p6rt
Copy link

p6rt commented May 2, 2012

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

Searchable as RT112728$

@p6rt
Copy link
Author

p6rt commented May 2, 2012

From @bbkr

17​:54 bbkr​: rakudo​: :2()
17​:54 p6eval​: rakudo 0059aa​: OUTPUT«Nominal type check failed for parameter '$str'; expected Str but got Parcel insteadâ�¤ in sub unbase at src/gen/CORE.setting​:4144â�¤ in block <anon> at /tmp/OzRQ79NLFt​:1â�¤â�¤Â»

IMO it should complaint about missing param, not about bad type

@p6rt
Copy link
Author

p6rt commented Dec 27, 2014

From @peschwa

Am Mi 02. Mai 2012, 09​:03​:34, pawel.pabian@​implix.com schrieb​:

17​:54 bbkr​: rakudo​: :2()
17​:54 p6eval​: rakudo 0059aa​: OUTPUT«Nominal type check failed for
parameter '$str'; expected Str but got Parcel instead� in sub unbase
at src/gen/CORE.setting​:4144â�¤ in block <anon> at /tmp/OzRQ79NLFt​:1â�¤â�¤Â»

IMO it should complaint about missing param, not about bad type

Rakudo currently throws an X​::Numeric​::Confused for the example code​:

13​:29 psch m​: :2() # RT #​112728
13​:29 camelia rakudo-moar 7ea485​: OUTPUT«You have confused the number with the textual representation "";â�¤if you wanted to render the number in the given base, use $number.base($radix)â�¤ in block <unit> at /tmp/na_4apNiYp​:1â�¤â�¤Â»

STD on the other hand parses it just fine, which I assume happens because there's no backend that actually does anything with the parse result​:

13​:29 psch std​: :2()
13​:29 camelia std 76ccee1​: OUTPUT«ok 00​:00 136mâ�¤Â»

@p6rt
Copy link
Author

p6rt commented Dec 27, 2014

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

@p6rt
Copy link
Author

p6rt commented Aug 29, 2015

From @coke

On Sat Dec 27 05​:33​:02 2014, peschwa@​gmail.com wrote​:

Am Mi 02. Mai 2012, 09​:03​:34, pawel.pabian@​implix.com schrieb​:

17​:54 bbkr​: rakudo​: :2()
17​:54 p6eval​: rakudo 0059aa​: OUTPUT«Nominal type check failed for
parameter '$str'; expected Str but got Parcel instead� in sub unbase
at src/gen/CORE.setting​:4144â�¤ in block <anon> at
/tmp/OzRQ79NLFt​:1â�¤â�¤Â»

IMO it should complaint about missing param, not about bad type

Rakudo currently throws an X​::Numeric​::Confused for the example code​:

13​:29 psch m​: :2() # RT #​112728
13​:29 camelia rakudo-moar 7ea485​: OUTPUT«You have confused
the number with the textual representation "";�if you wanted to
render the number in the given base, use $number.base($radix)� in
block <unit> at /tmp/na_4apNiYp​:1â�¤â�¤Â»

STD on the other hand parses it just fine, which I assume happens
because there's no backend that actually does anything with the parse
result​:

13​:29 psch std​: :2()
13​:29 camelia std 76ccee1​: OUTPUT«ok 00​:00 136mâ�¤Â»

Passing tests added to S02-literals/radix.t , resolving ticket.
--
Will "Coke" Coleda

@p6rt p6rt closed this as completed Aug 30, 2015
@p6rt
Copy link
Author

p6rt commented Aug 30, 2015

@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