-
Notifications
You must be signed in to change notification settings - Fork 210
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
fix: wait_for_all_recoveries #4496
Conversation
fedimint-client/src/lib.rs
Outdated
@@ -1491,7 +1491,7 @@ impl Client { | |||
.wait_for(|in_progress| { | |||
!in_progress |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this is hard, my head hurts
!in_progress | |
in_progress |
41f5384
to
5f4e29f
Compare
There's only so many combinations, we'll get it right eventually. |
in_progress | ||
.iter() | ||
.any(|(_id, progress)| !progress.is_done()) | ||
.all(|(_id, progress)| progress.is_done()) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This only improves readability, there is no functional change here.
To say it with some math:
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There's only so many combinations, we'll get it right eventually.
lol we are repeating combinations now
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lmaoooo
No description provided.