On Tue, 9 Aug 2022 at 02:41, Brian C. Lane <[email protected]> wrote: > +/* is_skip_guid checks the guid against the list of guids that should not be > + * overridden by set_system. It returns a 1 if it is in the list. > +*/ > +static bool > +is_skip_guid(efi_guid_t guid) { > + int n = sizeof(skip_set_system_guids) / sizeof(skip_set_system_guids[0]); > + for (int i = 0; i < n; ++i) { > + if (guid_cmp(guid, skip_set_system_guids[i]) == 0) { > + return 1; > + } > + } > + > + return 0; > +}
I think a function with return type bool rather should return true/false than 1/0.
