refactor(gstd): use waker-fn instead of unsafe code (#4271)

gear-tech · Oct 2, 2024 · f717377 · f717377
1 parent 80d46f4
commit f717377
Show file tree

Hide file tree

Showing 5 changed files with 5 additions and 41 deletions.
diff --git a/Cargo.lock b/Cargo.lock
diff --git a/gstd/Cargo.toml b/gstd/Cargo.toml
@@ -25,6 +25,7 @@ hex = { workspace = true, features = ["alloc"] }
 parity-scale-codec = { workspace = true, features = ["derive"] }
 scale-info = { workspace = true, features = ["derive"] }
 futures = { workspace = true, features = ["alloc"] }
+waker-fn = "1.2.0"
 
 [features]
 #! ## Default features

diff --git a/gstd/src/async_runtime/futures.rs b/gstd/src/async_runtime/futures.rs
@@ -47,7 +47,7 @@ impl Task {
         F: Future<Output = ()> + 'static,
     {
         Self {
-            waker: super::waker::empty(),
+            waker: waker_fn::waker_fn(|| {}),
             future: future.boxed_local(),
             lock_exceeded: false,
         }

diff --git a/gstd/src/async_runtime/mod.rs b/gstd/src/async_runtime/mod.rs
@@ -19,7 +19,6 @@
 mod futures;
 mod locks;
 mod signals;
-mod waker;
 
 #[cfg(not(feature = "ethexe"))]
 mod reply_hooks;

diff --git a/gstd/src/async_runtime/waker.rs b/gstd/src/async_runtime/waker.rs