I think you misinterpreted the Coding Standard. It says that while comparisons must be explicit, BOOLEAN variables are the exception and don't have to be compared to TRUE or FALSE. It is OPTIONAL whether or not the BOOLEAN variable is explicitly compared.
Explicit comparisons are always the most clear. It is the developer's choice whether, or not, to compare BOOLEAN variables explicitly. So, while the change is superfluous, I don't see anything wrong with it. Reviewed-by: Daryl McDaniel <[email protected]> Daryl -----Original Message----- From: Qiu Shumin [mailto:[email protected]] Sent: Friday, June 19, 2015 1:43 AM To: [email protected] Subject: [edk2] [PATCH] ShellPkg\Application\Shell: Refine the code style. BOOLEAN variable should not use explicit comparisons to TRUE or FALSE. To follow EDKII code style. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Qiu Shumin <[email protected]> --- ShellPkg/Application/Shell/ConsoleLogger.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/ShellPkg/Application/Shell/ConsoleLogger.c b/ShellPkg/Application/Shell/ConsoleLogger.c index 22abf55..9b64303 100644 --- a/ShellPkg/Application/Shell/ConsoleLogger.c +++ b/ShellPkg/Application/Shell/ConsoleLogger.c @@ -416,7 +416,7 @@ ConsoleLoggerReset ( // if (!EFI_ERROR (Status)) { ConsoleLoggerResetBuffers(ConsoleInfo); - if (ExtendedVerification == TRUE) { + if (ExtendedVerification) { ConsoleInfo->OriginalStartRow = 0; ConsoleInfo->CurrentStartRow = 0; } -- 1.9.5.msysgit.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
