Commit 58a623a

committed

Don't ask for aborting panics, make sure they're used

1 parent efacc77 commit 58a623aCopy full SHA for 58a623a

File tree

-4

lines changed

-4

lines changed

Lines changed: 6 additions & 0 deletions

Lines changed: 1 addition & 4 deletions

Original file line number	Diff line number	Diff line change
`@@ -227,9 +227,6 @@ impl BootServices {`
`227`	`227`	`/// priority level. If the event type has flag NotifyWait, the notification`
`228`	`228`	/// will be delivered next time `wait_for_event` or `check_event` is called.
`229`	`229`	/// In both cases, a `notify_fn` callback must be specified.
`230`		`- ///`
`231`		`- /// Note that the safety of this function relies on aborting panics being`
`232`		`- /// used, as requested by the uefi-rs documentation.`
`233`	`230`	`pub fn create_event(`
`234`	`231`	`&self,`
`235`	`232`	`event_ty: EventType,`
`@@ -242,7 +239,7 @@ impl BootServices {`
`242`	`239`	`// Use a trampoline to handle the impedance mismatch between Rust & C`
`243`	`240`	`unsafe extern "win64" fn notify_trampoline(e: Event, ctx: *mut c_void) {`
`244`	`241`	`let notify_fn: fn(Event) = core::mem::transmute(ctx);`
`245`		`- notify_fn(e); // Aborting panics are assumed here`
	`242`	`+ notify_fn(e); // SAFETY: Aborting panics are assumed here`
`246`	`243`	`}`
`247`	`244`	`let (notify_func, notify_ctx) = notify_fn`
`248`	`245`	`.map(\|notify_fn\| {`

Comments

(0)