mattwparas / steel

An embedded scheme interpreter in Rust
Apache License 2.0
1.05k stars 49 forks source link

Support comparison operations for BigNum. #153

Closed wmedrano closed 6 months ago

mattwparas commented 6 months ago

ready to merge?

wmedrano commented 6 months ago

Seems to work fine on the repl and the unit tests. I also noticed there were some number functions in vm.rs. Not sure how it is used. Should I update this file as well?

https://github.com/mattwparas/steel/blob/master/crates/steel-core/src/steel_vm/vm.rs#L5352

mattwparas commented 6 months ago

That is currently dead code and due for some cleanup - you can safely ignore that for now