Skip to content

Commit

Permalink
Patch use of set_precision on i586.
Browse files Browse the repository at this point in the history
Closes #218.
  • Loading branch information
Alexhuszagh committed Feb 6, 2025
1 parent 5af823f commit f190202
Show file tree
Hide file tree
Showing 3 changed files with 4 additions and 1 deletion.
2 changes: 2 additions & 0 deletions .github/workflows/Cross.yml
Original file line number Diff line number Diff line change
Expand Up @@ -24,6 +24,8 @@ jobs:
- aarch64-unknown-linux-gnu
- arm-unknown-linux-gnueabi
- armv7-unknown-linux-gnueabihf
# NOTE: Required for issue #218
- i586-unknown-linux-gnu
- i686-unknown-linux-gnu
- i686-unknown-linux-musl
# NOTE: This fails on cross v0.2.1, which is unusual since
Expand Down
1 change: 1 addition & 0 deletions CHANGELOG
Original file line number Diff line number Diff line change
Expand Up @@ -29,6 +29,7 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0

### Fixed

- Float parsing on `i586` targets (#218).
- Bug where the `radix` feature wasn't enabling `power-of-two` in `lexical-core` or `lexical` (#204).
- Fixed performance issues due to a lack of inlining on the Eisel-Lemire algorithm (#210).
- Issue with parsing non-decimal exponent radixes when using a decimal mantissa radix for floating-point numbers (#212).
Expand Down
2 changes: 1 addition & 1 deletion lexical-parse-float/src/number.rs
Original file line number Diff line number Diff line change
Expand Up @@ -74,7 +74,7 @@ impl Number<'_> {
// function takes care of setting the precision on architectures which
// require setting it by changing the global state (like the control word of the
// x87 FPU).
let _cw: () = set_precision::<F>();
let _cw = set_precision::<F>();

if self.is_fast_path::<F, FORMAT>() {
let radix = format.radix();
Expand Down

0 comments on commit f190202

Please sign in to comment.