Skip to content

Conversation

@djc
Copy link
Member

@djc djc commented Oct 30, 2025

No description provided.

@djc
Copy link
Member Author

djc commented Nov 1, 2025

@matheus23 do you know what might be up with the WASM breakage here? I suppose it is related to the 1.91 upgrade somehow?

@matheus23
Copy link
Contributor

Taking a look 👀

@matheus23
Copy link
Contributor

Yeah indeed, I can reproduce this failure in 1.91, but not in 1.89.
And it's Instant::checked_sub which fails in Wasm.
Not sure why the rust version has an effect on this, given it should run the same web_time::Instant code in both cases.
I'll keep looking.

@matheus23
Copy link
Contributor

This now also fails for me in 1.89. I'm debugging the values: now: Instant(15.746652507s) loss_delay: 17.246250153s.
So this is running into basically a version of rust-lang/rust#100141

I think this can be an issue in other platforms as well (not only Wasm, e.g. Mac, as the issue above suggests), as the Instant isn't guaranteed to have sufficient precision for the subtraction.

Anyway, we need to avoid .unwrap()ing here. I've opened #2436 to fix this issue

@djc djc merged commit 84d392c into main Nov 3, 2025
18 of 19 checks passed
@djc djc deleted the clippy-1.91 branch November 3, 2025 11:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants