On 2/23/24 03:33, Ni, Ray wrote: >> + ASSERT (FALSE); > > How about ASSERT (EFI_NOT_FOUND)? Which is more meaningful than FALSE.
I'll edit the patch -- assuming you mean ASSERT_EFI_ERROR (EFI_NOT_FOUND); Laszlo > > No matter you change or not, Reviewed-by: Ray Ni <ray...@intel.com> > >> + return 0; >> } >> >> /** >> -- >> 2.43.2 > > > > > > -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#115964): https://edk2.groups.io/g/devel/message/115964 Mute This Topic: https://groups.io/mt/104510913/21656 Group Owner: devel+ow...@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-