mattwparas / steel / 15315045212

45%

Build Type

push

github

Committed by web-flow

Commit Message

optimize real_literal_to_steelval and deduplicate number_literal_to_steel (#405)

* lightly optimize real_literal_to_steelval

avoid an allocation when converting a RealLiteral::Rational to a
SteelVal::Rational.

also avoid panicking when the denominator is 0

* move number_literal_to_steel into an impl IntoSteelval for &NumberLiteral

* deduplicate IntoSteelVal for &NumberLiteral

as we already need to clone out all of the BigInts, we can just clone
them first and then delegate to the impl for NumberLiteral

Run Details

11 of 14 new or added lines in 5 files covered. (78.57%)

15 existing lines in 7 files now uncovered.

12492 of 27162 relevant lines covered (45.99%)

454874.47 hits per line