Skip to content

Commit 988575d

Browse files
authored
Merge pull request #219 from Alexhuszagh/issue_218
Patch use of `set_precision` on i586.
2 parents 0d9e161 + 709e6d7 commit 988575d

File tree

3 files changed

+4
-1
lines changed

3 files changed

+4
-1
lines changed

.github/workflows/Cross.yml

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -24,6 +24,8 @@ jobs:
2424
- aarch64-unknown-linux-gnu
2525
- arm-unknown-linux-gnueabi
2626
- armv7-unknown-linux-gnueabihf
27+
# NOTE: Required for issue #218
28+
- i586-unknown-linux-gnu
2729
- i686-unknown-linux-gnu
2830
- i686-unknown-linux-musl
2931
# NOTE: This fails on cross v0.2.1, which is unusual since

CHANGELOG

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -31,6 +31,7 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
3131

3232
- Patch doctests with TODOs in dependencies (#222).
3333
- CI with OSS-Fuzz for the deprecated `actions/upload-artifact@v3` (#221).
34+
- Float parsing on `i586` targets (#219).
3435
- Bug where the `radix` feature wasn't enabling `power-of-two` in `lexical-core` or `lexical` (#204).
3536
- Fixed performance issues due to a lack of inlining on the Eisel-Lemire algorithm (#210).
3637
- Issue with parsing non-decimal exponent radixes when using a decimal mantissa radix for floating-point numbers (#212).

lexical-parse-float/src/number.rs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -74,7 +74,7 @@ impl Number<'_> {
7474
// function takes care of setting the precision on architectures which
7575
// require setting it by changing the global state (like the control word of the
7676
// x87 FPU).
77-
let _cw: () = set_precision::<F>();
77+
let _cw = set_precision::<F>();
7878

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

0 commit comments

Comments
 (0)