nordic-dev.net/embassy
2
0
Fork 0
mirror of https://github.com/embassy-rs/embassy.git synced 2024-11-21 22:32:29 +00:00
Code Issues Packages Projects Releases Wiki Activity

executor: compare vtable addr instead of contents.

Browse Source 
Saves a whopping 44 bytes of text, yay.
This commit is contained in:
Dario Nieuwenhuis 2024-11-12 16:30:46 +01:00
parent baeb59b5b8
commit 853c5c567a
1 changed files with 3 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
embassy-executor/src/raw/waker.rs
View File

@ -43,7 +43,9 @@ pub fn task_from_waker(waker: &Waker) -> TaskRef {
// TODO use waker_getters when stable. https://github.com/rust-lang/rust/issues/96992 // TODO use waker_getters when stable. https://github.com/rust-lang/rust/issues/96992
let hack: &WakerHack = unsafe { core::mem::transmute(waker) }; let hack: &WakerHack = unsafe { core::mem::transmute(waker) };
if hack.vtable != &VTABLE { // make sure to compare vtable addresses. Doing `==` on the references
// will compare the contents, which is slower.
if hack.vtable as *const _ != &VTABLE as *const _ {
panic!("Found waker not created by the Embassy executor. `embassy_time::Timer` only works with the Embassy executor.") panic!("Found waker not created by the Embassy executor. `embassy_time::Timer` only works with the Embassy executor.")
} }
// safety: our wakers are always created with `TaskRef::as_ptr` // safety: our wakers are always created with `TaskRef::as_ptr`