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.new.Str causes Null PMC #1268

Closed
p6rt opened this issue Sep 2, 2009 · 5 comments
Closed

Rat.new.Str causes Null PMC #1268

p6rt opened this issue Sep 2, 2009 · 5 comments
Labels
Bug LHF Low-hanging fruit, easy to resolve

Comments

@p6rt
Copy link

p6rt commented Sep 2, 2009

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

Searchable as RT68958$

@p6rt
Copy link
Author

p6rt commented Sep 2, 2009

From @colomon

colomon​: rakudo​: say Rat.new.Str
p6eval​: rakudo c9a930​: OUTPUT«Null PMC access in find_method()␤in
method Rat​::Num (src/gen_setting.pm​:993)␤called from method Rat​::Str
(src/gen_setting.pm​:995)␤called from Main (/tmp/igcBl8SLmF​:2)␤»

I've got no idea what that is supposed to do, but am reporting this
under the principle that Null PMC's are always bug-worthy.

--
Solomon Foster​: colomon@​gmail.com
HarmonyWare, Inc​: http://www.harmonyware.com

@p6rt
Copy link
Author

p6rt commented Apr 19, 2010

From @jnthn

On Wed Sep 02 13​:06​:46 2009, colomon@​gmail.com wrote​:

colomon​: rakudo​: say Rat.new.Str
p6eval​: rakudo c9a930​: OUTPUT«Null PMC access in find_method()␤in
method Rat​::Num (src/gen_setting.pm​:993)␤called from method Rat​::Str
(src/gen_setting.pm​:995)␤called from Main (/tmp/igcBl8SLmF​:2)␤»

I've got no idea what that is supposed to do, but am reporting this
under the principle that Null PMC's are always bug-worthy.

Now does​:

01​:01 <@​jnthn> rakudo​: say Rat.new.Str
01​:01 <+p6eval> rakudo 346e76​: OUTPUT«0␤»

Which is probably sensible. Given to moritz++ for spectests.

Thanks,

Jonathan

@p6rt
Copy link
Author

p6rt commented Apr 19, 2010

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

@p6rt
Copy link
Author

p6rt commented Apr 22, 2010

From @moritz

Now tested in t/spec/S32-num/rat.t, and works.

@p6rt
Copy link
Author

p6rt commented Apr 22, 2010

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

@p6rt p6rt closed this as completed Apr 22, 2010
@p6rt p6rt added Bug LHF Low-hanging fruit, easy to resolve labels Jan 5, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug LHF Low-hanging fruit, easy to resolve
Projects
None yet
Development

No branches or pull requests

1 participant