On Mon, Nov 11, 2019 at 08:24:43PM +0000, Andrew Cooper wrote: > actypes.h and efidef.h both define BOOLEAN as unsigned char, which is buggy in > combination with logic such as "BOOLEAN b = (a & 0x100);" Redefine BOOLEAN as > bool instead, which doesn't truncate. > > Both also define TRUE and FALSE, with actypes.h being extra rude and replacing > whatever exists thus far. Drop all uses of TRUE and FALSE, replacing them > with true/false respectively, and drop the declarations. > > Also drop the pointless conditional declaration of NULL while cleaning this > up. > > Finally, correct all the comments which which were found by sed. > > Signed-off-by: Andrew Cooper <[email protected]>
Reviewed-by: Wei Liu <[email protected]> _______________________________________________ Xen-devel mailing list [email protected] https://lists.xenproject.org/mailman/listinfo/xen-devel
