Albin Hedman
Albin Hedman
In my local fork of this repo I have converted from 0.1 futures and tokio 0.1 to std futures with async await and tokio 0.2 . However this depends on...
I'm still waiting for a review
Sorry for late response. I can not speak for @knsd but to me a fork sounds like a plan :) What about just `async_ping`?
> I dislike that a bit, because async_ping sounds runtime agnostic to me, which this is very much not. That is fair. > Considering we're (on a lower level) using...
> And done: https://crates.io/crates/tokio-icmp-echo. It's available under https://github.com/jcgruenhage/tokio-icmp-echo. @usbalbin I've invited you as a collaborator on the repo and on the crate on crates.io I can not promise how much...
I just opened #15 which changes to new tokio and futures. This removes the need for `Handle::default()`, not sure if this affects your problem in any way. Thought I'd mention...
If I am not mistaken I believe at one point I had a problem where I accedentally dropped the pinger before the future resolved. This caused some similar problem where...
@agrinman oh, I almost forgot about this PR. I sort of turned my focus towards #14 (rebase of #12) which is focused on the edition change. Therefore I kind of...
> @usbalbin this PR looks outdated and there are more non-necessary stuff. I am planning on getting back to this at some point early next year. Will remove all unrelated...
To me I think it would have made more sense if the uart and dma stream were combined into a "uart-with-dma" rather than a "dma-stream-thing that happens to wrap an...