On 05/27/15 20:32, Jordan Justen wrote: > On 2015-05-27 08:32:52, Olivier Martin wrote: >> ARM toolchain raises the build error: "enumerated >> type mixed with another type" >> >> Contributed-under: TianoCore Contribution Agreement 1.0 >> Signed-off-by: Olivier Martin <[email protected]> >> --- >> MdeModulePkg/Core/Dxe/Mem/MemoryProfileRecord.c | 2 +- >> 1 file changed, 1 insertion(+), 1 deletion(-) >> >> diff --git a/MdeModulePkg/Core/Dxe/Mem/MemoryProfileRecord.c >> b/MdeModulePkg/Core/Dxe/Mem/MemoryProfileRecord.c >> index 2c6713f..b23a056 100644 >> --- a/MdeModulePkg/Core/Dxe/Mem/MemoryProfileRecord.c >> +++ b/MdeModulePkg/Core/Dxe/Mem/MemoryProfileRecord.c >> @@ -789,7 +789,7 @@ GetProfileMemoryIndex ( >> if ((UINT32) MemoryType >= MEMORY_TYPE_OS_RESERVED_MIN) { >> return EfiMaxMemoryType; >> } else if ((UINT32) MemoryType >= MEMORY_TYPE_OEM_RESERVED_MIN) { >> - return EfiMaxMemoryType + 1; >> + return (EFI_MEMORY_TYPE)(EfiMaxMemoryType + 1); > > The code style would want a space after the cast close parens, right?
(It would, and it is so wrong! Obligatory reference: <https://github.com/tianocore/edk2/commit/71914406>. This is the one and only bit in the code style that is entirely harmful.) Laszlo > > Reviewed-by: Jordan Justen <[email protected]> > >> } else { >> return MemoryType; >> } >> -- >> 2.1.1 >> >> >> ------------------------------------------------------------------------------ >> _______________________________________________ >> edk2-devel mailing list >> [email protected] >> https://lists.sourceforge.net/lists/listinfo/edk2-devel > > ------------------------------------------------------------------------------ > _______________________________________________ > edk2-devel mailing list > [email protected] > https://lists.sourceforge.net/lists/listinfo/edk2-devel > ------------------------------------------------------------------------------ _______________________________________________ edk2-devel mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/edk2-devel
