search

knurling-rs / defmt

Efficient, deferred formatting for logging on embedded systems
https://defmt.ferrous-systems.com/
Apache License 2.0
812 stars 74 forks source link

Make defmt-print async #855

Closed JomerDev closed 3 months ago

JomerDev commented 3 months ago

As discussed in #807 I've split out the work to make defmt-print async into a separate PR. It changes very little, but please do test it to see if everything still works. As for the tokio features, I've added all of the features needed instead of using full but afaik it wouldn't have made much of a difference as most features were needed anyway.

jonathanpallant commented 3 months ago

Build time (main): 57.09s user 2.33s system 215% cpu 27.534 total

Build time (PR): 69.60s user 2.75s system 226% cpu 31.895 total

So an extra 4 seconds to compile, or +15%, on this macBook. I think that's fine.