async-broadcast icon indicating copy to clipboard operation
async-broadcast copied to clipboard

Published 20 hours ago • verified publisher icon smol-rs

adding `no_std` support

Open dscso opened this issue 9 months ago • 3 comments

It seams to pass all tests locally and also runs on the ESP32 as far as I see. it seams to address #50 as well. I am not that familiar with wasm tho.

dscso avatar May 02 '24 13:05 dscso

It seams to pass all tests locally and also runs on the ESP32 as far as I see. it seams to address #50 as well.

Nice! Thanks for this PR. I'm sorry I didn't get to review it yet. I just started a new job so a bit busy with that. :) I'll try to have a look soon.

zeenix avatar May 03 '24 17:05 zeenix

Cool! Thanks for the clarification and no worries there is no need to rush. I hope you can also use Rust in your Job :D

dscso avatar May 03 '24 19:05 dscso

Cool! Thanks for the clarification and no worries there is no need to rush. I hope you can also use Rust in your Job :D

I wouldn't take the job, otherwise. :)

zeenix avatar May 05 '24 11:05 zeenix

@dscso Ping. Do you think you can finish this soon? We're hoping to do a 1.0 soon and it'd be great to have this done before that.

zeenix avatar Jun 08 '24 19:06 zeenix

I'd like to avoid using spinlocks in concurrent code.

I also think spinlocks are not the way to solve the issue. In my setting I was using the library in a single processor no interrupt setting, and there spin locks would be valid a solution.

I don't have the resources to implement a complete and clean no_std version without spinlocks.

dscso avatar Jun 10 '24 10:06 dscso

I don't have the resources to implement a complete and clean no_std version without spinlocks.

Fair enough. Thanks so much for your efforts here, still! I learnt a lot from the PR so it was useful even it it didn't get merged. :+1:

zeenix avatar Jun 10 '24 10:06 zeenix