Reviewed-by: Bi Dandan <[email protected]> Thanks, Dandan
> -----Original Message----- > From: Wang, Jian J > Sent: Saturday, October 27, 2018 10:13 AM > To: [email protected] > Cc: Bi, Dandan <[email protected]>; Zeng, Star <[email protected]> > Subject: [PATCH] MdeModulePkg/Core: correct one coding style > > REF: https://bugzilla.tianocore.org/show_bug.cgi?id=1284 > > Non-Boolean comparisons should use a compare operator (==, !=, >, < >=, <=) > > Cc: Dandan Bi <[email protected]> > Cc: Star Zeng <[email protected]> > Contributed-under: TianoCore Contribution Agreement 1.1 > Signed-off-by: Jian J Wang <[email protected]> > --- > MdeModulePkg/Core/Dxe/Mem/HeapGuard.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/MdeModulePkg/Core/Dxe/Mem/HeapGuard.c > b/MdeModulePkg/Core/Dxe/Mem/HeapGuard.c > index 521e0d7b2a..9f89df8d8f 100644 > --- a/MdeModulePkg/Core/Dxe/Mem/HeapGuard.c > +++ b/MdeModulePkg/Core/Dxe/Mem/HeapGuard.c > @@ -1559,7 +1559,7 @@ PromoteGuardedFreePages ( > } > } > > - if (AvailablePages) { > + if (AvailablePages != 0) { > DEBUG ((DEBUG_INFO, "Promoted pages: %lX (%lx)\r\n", Start, > (UINT64)AvailablePages)); > ClearGuardedMemoryBits (Start, AvailablePages); > > -- > 2.19.0.windows.1 _______________________________________________ edk2-devel mailing list [email protected] https://lists.01.org/mailman/listinfo/edk2-devel
