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

No case for ToDoubleNodeGen node with values of type BigDecimal #2815

Closed
gogainda opened this issue Dec 28, 2022 · 2 comments
Closed

No case for ToDoubleNodeGen node with values of type BigDecimal #2815

gogainda opened this issue Dec 28, 2022 · 2 comments

Comments

@gogainda
Copy link
Contributor

gogainda commented Dec 28, 2022

Found here

ruby -e "require 'bigdecimal'; puts [BigDecimal('123456789.123456789123456789')].pack('G')"


-e:1:in `pack': TruffleRuby doesn't have a case for the org.truffleruby.core.format.convert.ToDoubleNodeGen node with values of type BigDecimal(org.truffleruby.core.basicobject.RubyBasicObject) (TypeError)
	from org.truffleruby.core.format.convert.ToDoubleNodeGen.executeAndSpecialize(ToDoubleNodeGen.java:136)
	from org.truffleruby.core.format.convert.ToDoubleNodeGen.executeToDouble(ToDoubleNodeGen.java:55)
	from org.truffleruby.core.format.read.array.ReadDoubleNode.read(ReadDoubleNode.java:33)
	from org.truffleruby.core.format.read.array.ReadDoubleNodeGen.executeAndSpecialize(ReadDoubleNodeGen.java:100)
	from org.truffleruby.core.format.read.array.ReadDoubleNodeGen.execute(ReadDoubleNodeGen.java:64)
	from org.truffleruby.core.format.convert.ReinterpretAsLongNodeGen.execute(ReinterpretAsLongNodeGen.java:27)
	from org.truffleruby.core.format.convert.Integer64BigToBytesNodeGen.execute(Integer64BigToBytesNodeGen.java:27)
	from org.truffleruby.core.format.write.bytes.WriteBytesNodeGen.execute(WriteBytesNodeGen.java:31)
	from org.truffleruby.core.format.FormatRootNode.execute(FormatRootNode.java:57)
	from -e:1:in `<main>'
@andrykonchin
Copy link
Member

Thank you for reporting!

@andrykonchin
Copy link
Member

Fixed in 078b972

@andrykonchin andrykonchin added this to the 23.0.0 Release milestone Jan 4, 2023
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

2 participants