Update the code to following EDK coding style document.

1) Pointer value should compare with NULL.
2) Integer should compare with 0.
3) BOOLEAN should not compare with TRUE or FALSE.

git-svn-id: https://edk2.svn.sourceforge.net/svnroot/edk2/trunk/edk2@5413 6f19259b-4bc3-4df7-8a09-765794883524

11 files changed, 225 insertions, 225 deletions

diff --git a/MdeModulePkg/Universal/BdsDxe/BootMaint/BBSsupport.c b/MdeModulePkg/Universal/BdsDxe/BootMaint/BBSsupport.c
index ab3b24b2bd..24c9342008 100644
--- a/MdeModulePkg/Universal/BdsDxe/BootMaint/BBSsupport.c
+++ b/MdeModulePkg/Universal/BdsDxe/BootMaint/BBSsupport.c
@@ -92,7 +92,7 @@ UnicodeToAscii (
 }

 

 /**

-  EDES_TODO: Add function description

+  EDES_TODO: Add function description.

 

   @param CurBBSEntry     EDES_TODO: Add parameter description

   @param Index           EDES_TODO: Add parameter description

@@ -393,7 +393,7 @@ BdsCreateLegacyBootOption (
 }

 

 /**

-  EDES_TODO: Add function description

+  EDES_TODO: Add function description.

 

   @param BootOptionVar   EDES_TODO: Add parameter description

   @param BbsEntry        EDES_TODO: Add parameter description

@@ -435,7 +435,7 @@ BdsIsLegacyBootOption (
 }

 

 /**

-  EDES_TODO: Add function description

+  EDES_TODO: Add function description.

 

   @param OptionNumber    EDES_TODO: Add parameter description

   @param BootOrder       EDES_TODO: Add parameter description

@@ -606,7 +606,7 @@ BdsDeleteAllInvalidLegacyBootOptions (
       );

   }

 

-  if (BootOrderSize) {

+  if (BootOrderSize != 0) {

     Status = gRT->SetVariable (

                     L"BootOrder",

                     &gEfiGlobalVariableGuid,

@@ -624,7 +624,7 @@ BdsDeleteAllInvalidLegacyBootOptions (
 }

 

 /**

-  EDES_TODO: Add function description

+  EDES_TODO: Add function description.

 

   @param BootOrder       EDES_TODO: Add parameter description

   @param BootOptionNum   EDES_TODO: Add parameter description

@@ -694,7 +694,7 @@ BdsFindLegacyBootOptionByDevType (
 }

 

 /**

-  EDES_TODO: Add function description

+  EDES_TODO: Add function description.

 

   @param BbsItem         EDES_TODO: Add parameter description

   @param Index           EDES_TODO: Add parameter description

@@ -858,7 +858,7 @@ BdsAddNonExistingLegacyBootOptions (
 }

 

 /**

-  EDES_TODO: Add function description

+  EDES_TODO: Add function description.

 

   @param BbsTable        EDES_TODO: Add parameter description

   @param BbsType         EDES_TODO: Add parameter description

@@ -895,7 +895,7 @@ BdsFillDevOrderBuf (
 }

 

 /**

-  EDES_TODO: Add function description

+  EDES_TODO: Add function description.

 

   @param BbsTable        EDES_TODO: Add parameter description

   @param BbsCount        EDES_TODO: Add parameter description

@@ -980,7 +980,7 @@ BdsCreateDevOrder (
   Ptr += sizeof (BBS_TYPE);

   *((UINT16 *) Ptr) = (UINT16) (sizeof (UINT16) + FDCount * sizeof (UINT16));

   Ptr += sizeof (UINT16);

-  if (FDCount) {

+  if (FDCount != 0) {

     Ptr = (UINT8 *) BdsFillDevOrderBuf (BbsTable, BBS_FLOPPY, BbsCount, (UINT16 *) Ptr);

   }

 

@@ -988,7 +988,7 @@ BdsCreateDevOrder (
   Ptr += sizeof (BBS_TYPE);

   *((UINT16 *) Ptr) = (UINT16) (sizeof (UINT16) + HDCount * sizeof (UINT16));

   Ptr += sizeof (UINT16);

-  if (HDCount) {

+  if (HDCount != 0) {

     Ptr = (UINT8 *) BdsFillDevOrderBuf (BbsTable, BBS_HARDDISK, BbsCount, (UINT16 *) Ptr);

   }

 

@@ -996,7 +996,7 @@ BdsCreateDevOrder (
   Ptr += sizeof (BBS_TYPE);

   *((UINT16 *) Ptr) = (UINT16) (sizeof (UINT16) + CDCount * sizeof (UINT16));

   Ptr += sizeof (UINT16);

-  if (CDCount) {

+  if (CDCount != 0) {

     Ptr = (UINT8 *) BdsFillDevOrderBuf (BbsTable, BBS_CDROM, BbsCount, (UINT16 *) Ptr);

   }

 

@@ -1004,7 +1004,7 @@ BdsCreateDevOrder (
   Ptr += sizeof (BBS_TYPE);

   *((UINT16 *) Ptr) = (UINT16) (sizeof (UINT16) + NETCount * sizeof (UINT16));

   Ptr += sizeof (UINT16);

-  if (NETCount) {

+  if (NETCount != 0) {

     Ptr = (UINT8 *) BdsFillDevOrderBuf (BbsTable, BBS_EMBED_NETWORK, BbsCount, (UINT16 *) Ptr);

   }

 

@@ -1012,7 +1012,7 @@ BdsCreateDevOrder (
   Ptr += sizeof (BBS_TYPE);

   *((UINT16 *) Ptr) = (UINT16) (sizeof (UINT16) + BEVCount * sizeof (UINT16));

   Ptr += sizeof (UINT16);

-  if (BEVCount) {

+  if (BEVCount != 0) {

     Ptr = (UINT8 *) BdsFillDevOrderBuf (BbsTable, BBS_BEV_DEVICE, BbsCount, (UINT16 *) Ptr);

   }

 

@@ -1029,7 +1029,7 @@ BdsCreateDevOrder (
 }

 

 /**

-  EDES_TODO: Add function description

+  EDES_TODO: Add function description.

 

   @param VOID            EDES_TODO: Add parameter description

 

@@ -1334,7 +1334,7 @@ BdsUpdateLegacyDevOrder (
     // at this point we have copied those valid indexes to new buffer

     // and we should check if there is any new appeared boot device

     //

-    if (Idx) {

+    if (Idx != 0) {

       for (Index2 = 0; Index2 < *Idx; Index2++) {

         if ((NewDevPtr[Index2] & 0xFF) == (UINT16) Index) {

           break;

@@ -1353,7 +1353,7 @@ BdsUpdateLegacyDevOrder (
     }

   }

 

-  if (FDCount) {

+  if (FDCount != 0) {

     //

     // Just to make sure that disabled indexes are all at the end of the array

     //

@@ -1373,7 +1373,7 @@ BdsUpdateLegacyDevOrder (
     }

   }

 

-  if (HDCount) {

+  if (HDCount != 0) {

     //

     // Just to make sure that disabled indexes are all at the end of the array

     //

@@ -1393,7 +1393,7 @@ BdsUpdateLegacyDevOrder (
     }

   }

 

-  if (CDCount) {

+  if (CDCount != 0) {

     //

     // Just to make sure that disabled indexes are all at the end of the array

     //

@@ -1413,7 +1413,7 @@ BdsUpdateLegacyDevOrder (
     }

   }

 

-  if (NETCount) {

+  if (NETCount != 0) {

     //

     // Just to make sure that disabled indexes are all at the end of the array

     //

@@ -1433,7 +1433,7 @@ BdsUpdateLegacyDevOrder (
     }

   }

 

-  if (BEVCount) {

+  if (BEVCount!= 0) {

     //

     // Just to make sure that disabled indexes are all at the end of the array

     //

@@ -1468,7 +1468,7 @@ BdsUpdateLegacyDevOrder (
 }

 

 /**

-  EDES_TODO: Add function description

+  EDES_TODO: Add function description.

 

   @param DeviceType      EDES_TODO: Add parameter description

   @param LocalBbsTable   EDES_TODO: Add parameter description

@@ -1538,7 +1538,7 @@ BdsSetBootPriority4SameTypeDev (
 }

 

 /**

-  EDES_TODO: Add function description

+  EDES_TODO: Add function description.

 

   @param LocalBbsTable   EDES_TODO: Add parameter description

 

@@ -1586,7 +1586,7 @@ PrintBbsTable (
 }

 

 /**

-  EDES_TODO: Add function description

+  EDES_TODO: Add function description.

 

   @param Entry           EDES_TODO: Add parameter description

 

@@ -1671,7 +1671,7 @@ BdsRefreshBbsTableForBoot (
                           &gEfiGlobalVariableGuid,

                           &BootOrderSize

                           );

-  for (Index = 0; BootOrder && Index < BootOrderSize / sizeof (UINT16); Index++) {

+  for (Index = 0; ((BootOrder != NULL) && (Index < BootOrderSize / sizeof (UINT16))); Index++) {

     UnicodeSPrint (BootOption, sizeof (BootOption), L"Boot%04x", BootOrder[Index]);

     BootOptionVar = BdsLibGetVariableAndSize (

                       BootOption,

@@ -1714,7 +1714,7 @@ BdsRefreshBbsTableForBoot (
     }

   }

 

-  if (BootOrder) {

+  if (BootOrder != NULL) {

     SafeFreePool (BootOrder);

   }

   //

diff --git a/MdeModulePkg/Universal/BdsDxe/BootMaint/BBSsupport.h b/MdeModulePkg/Universal/BdsDxe/BootMaint/BBSsupport.h
index 1b32084450..ce60cd9391 100644
--- a/MdeModulePkg/Universal/BdsDxe/BootMaint/BBSsupport.h
+++ b/MdeModulePkg/Universal/BdsDxe/BootMaint/BBSsupport.h
@@ -20,7 +20,7 @@ WITHOUT WARRANTIES OR REPRESENTATIONS OF ANY KIND, EITHER EXPRESS OR IMPLIED.
 // Bugbug: Candidate for a PCD entries

 //

 /**

-  EDES_TODO: Add function description

+  EDES_TODO: Add function description.

 

   @param CurBBSEntry     EDES_TODO: Add parameter description

   @param Index           EDES_TODO: Add parameter description

@@ -41,7 +41,7 @@ BdsBuildLegacyDevNameString (
   );

 

 /**

-  EDES_TODO: Add function description

+  EDES_TODO: Add function description.

 

   @param VOID            EDES_TODO: Add parameter description

 

@@ -71,7 +71,7 @@ BdsAddNonExistingLegacyBootOptions (
 ;

 

 /**

-  EDES_TODO: Add function description

+  EDES_TODO: Add function description.

 

   @param VOID            EDES_TODO: Add parameter description

 

@@ -84,7 +84,7 @@ BdsUpdateLegacyDevOrder (
   );

 

 /**

-  EDES_TODO: Add function description

+  EDES_TODO: Add function description.

 

   @param Entry           EDES_TODO: Add parameter description

 

diff --git a/MdeModulePkg/Universal/BdsDxe/BootMaint/BmLib.c b/MdeModulePkg/Universal/BdsDxe/BootMaint/BmLib.c
index fbf2a0bfbe..d5c32c8ac1 100644
--- a/MdeModulePkg/Universal/BdsDxe/BootMaint/BmLib.c
+++ b/MdeModulePkg/Universal/BdsDxe/BootMaint/BmLib.c
@@ -145,7 +145,7 @@ EfiGrowBuffer (
   //

   // If this is an initial request, buffer will be null with a new buffer size

   //

-  if (!*Buffer && BufferSize) {

+  if ((*Buffer == NULL) && (BufferSize != 0)) {

     *Status = EFI_BUFFER_TOO_SMALL;

   }

   //

@@ -158,7 +158,7 @@ EfiGrowBuffer (
 

     *Buffer = EfiAllocateZeroPool (BufferSize);

 

-    if (*Buffer) {

+    if (*Buffer != NULL) {

       TryAgain = TRUE;

     } else {

       *Status = EFI_OUT_OF_RESOURCES;

@@ -167,7 +167,7 @@ EfiGrowBuffer (
   //

   // If there's an error, free the buffer

   //

-  if (!TryAgain && EFI_ERROR (*Status) && *Buffer) {

+  if (!TryAgain && EFI_ERROR (*Status) && (*Buffer != NULL)) {

     SafeFreePool (*Buffer);

     *Buffer = NULL;

   }

@@ -224,7 +224,7 @@ EfiLibDeleteVariable (
   VarBuf  = EfiLibGetVariable (VarName, VarGuid);

   Status  = EFI_NOT_FOUND;

 

-  if (VarBuf) {

+  if (VarBuf != NULL) {

     //

     // Delete variable from Storage

     //

@@ -296,7 +296,7 @@ EfiStrDuplicate (
   Size  = StrSize (Src);

   Dest  = EfiAllocateZeroPool (Size);

   ASSERT (Dest != NULL);

-  if (Dest) {

+  if (Dest != NULL) {

     CopyMem (Dest, Src, Size);

   }

 

@@ -393,12 +393,12 @@ EfiReallocatePool (
   VOID  *NewPool;

 

   NewPool = NULL;

-  if (NewSize) {

+  if (NewSize != 0) {

     NewPool = EfiAllocateZeroPool (NewSize);

   }

 

-  if (OldPool) {

-    if (NewPool) {

+  if (OldPool != NULL) {

+    if (NewPool != NULL) {

       CopyMem (NewPool, OldPool, OldSize < NewSize ? OldSize : NewSize);

     }

 

diff --git a/MdeModulePkg/Universal/BdsDxe/BootMaint/BootMaint.c b/MdeModulePkg/Universal/BdsDxe/BootMaint/BootMaint.c
index 9fca22afbd..6eae9c18dd 100644
--- a/MdeModulePkg/Universal/BdsDxe/BootMaint/BootMaint.c
+++ b/MdeModulePkg/Universal/BdsDxe/BootMaint/BootMaint.c
@@ -38,7 +38,7 @@ InitAllMenu (
   );

 

 /**

-  EDES_TODO: Add function description

+  EDES_TODO: Add function description.

 

   @param VOID            EDES_TODO: Add parameter description

 

@@ -723,7 +723,7 @@ Error:
 }

 

 /**

-  EDES_TODO: Add function description

+  EDES_TODO: Add function description.

 

   @param Private         EDES_TODO: Add parameter description

   @param CurrentFakeNVMap EDES_TODO: Add parameter description

@@ -812,7 +812,7 @@ InitializeBM (
   // Create LoadOption in BmmCallbackInfo for Driver Callback

   //

   Ptr = EfiAllocateZeroPool (sizeof (BM_LOAD_CONTEXT) + sizeof (BM_FILE_CONTEXT) + sizeof (BM_HANDLE_CONTEXT) + sizeof (BM_MENU_ENTRY));

-  if (!Ptr) {

+  if (Ptr == NULL) {

     SafeFreePool (BmmCallbackInfo);

     return EFI_OUT_OF_RESOURCES;

   }

@@ -1031,7 +1031,7 @@ InitializeBM (
 }

 

 /**

-  EDES_TODO: Add function description

+  EDES_TODO: Add function description.

 

   @param CallbackData    EDES_TODO: Add parameter description

 

@@ -1060,7 +1060,7 @@ InitAllMenu (
 }

 

 /**

-  EDES_TODO: Add function description

+  EDES_TODO: Add function description.

 

   @param VOID            EDES_TODO: Add parameter description

 

diff --git a/MdeModulePkg/Universal/BdsDxe/BootMaint/BootMaint.h b/MdeModulePkg/Universal/BdsDxe/BootMaint/BootMaint.h
index 2b8782944e..1ac2707c2f 100644
--- a/MdeModulePkg/Universal/BdsDxe/BootMaint/BootMaint.h
+++ b/MdeModulePkg/Universal/BdsDxe/BootMaint/BootMaint.h
@@ -470,7 +470,7 @@ typedef struct _STRING_DEPOSITORY {
 // For initializing File System menu

 //

 /**

-  EDES_TODO: Add function description

+  EDES_TODO: Add function description.

 

   @param CallbackData    EDES_TODO: Add parameter description

 

@@ -487,7 +487,7 @@ BOpt_FindFileSystem (
 // For cleaning up File System menu

 //

 /**

-  EDES_TODO: Add function description

+  EDES_TODO: Add function description.

 

   @param VOID            EDES_TODO: Add parameter description

 

@@ -504,7 +504,7 @@ BOpt_FreeFileSystem (
 // For initializing File Navigation menu

 //

 /**

-  EDES_TODO: Add function description

+  EDES_TODO: Add function description.

 

   @param CallbackData    EDES_TODO: Add parameter description

   @param MenuEntry       EDES_TODO: Add parameter description

@@ -523,7 +523,7 @@ BOpt_FindFiles (
 // For cleaning up File Navigation menu

 //

 /**

-  EDES_TODO: Add function description

+  EDES_TODO: Add function description.

 

   @param VOID            EDES_TODO: Add parameter description

 

@@ -540,7 +540,7 @@ BOpt_FreeFiles (
 // For Initializing handle navigation menu

 //

 /**

-  EDES_TODO: Add function description

+  EDES_TODO: Add function description.

 

   @param VOID            EDES_TODO: Add parameter description

 

@@ -557,7 +557,7 @@ BOpt_FindDrivers (
 // For Cleaning up handle navigation menu

 //

 /**

-  EDES_TODO: Add function description

+  EDES_TODO: Add function description.

 

   @param VOID            EDES_TODO: Add parameter description

 

@@ -571,7 +571,7 @@ BOpt_FreeDrivers(VOID);
 // For initializing Boot Option menu

 //

 /**

-  EDES_TODO: Add function description

+  EDES_TODO: Add function description.

 

   @param CallbackData    EDES_TODO: Add parameter description

 

@@ -587,7 +587,7 @@ BOpt_GetBootOptions (
 // For Initializing Driver option menu

 //

 /**

-  EDES_TODO: Add function description

+  EDES_TODO: Add function description.

 

   @param CallbackData    EDES_TODO: Add parameter description

 

@@ -603,7 +603,7 @@ BOpt_GetDriverOptions (
 // For Cleaning up boot option menu

 //

 /**

-  EDES_TODO: Add function description

+  EDES_TODO: Add function description.

 

   @param VOID            EDES_TODO: Add parameter description

 

@@ -617,7 +617,7 @@ BOpt_FreeBootOptions (VOID);
 // For cleaning up driver option menu

 //

 /**

-  EDES_TODO: Add function description

+  EDES_TODO: Add function description.

 

   @param VOID            EDES_TODO: Add parameter description

 

@@ -631,7 +631,7 @@ BOpt_FreeDriverOptions(VOID);
 // For Initializing HD/FD/CD/NET/BEV option menu

 //

 /**

-  EDES_TODO: Add function description

+  EDES_TODO: Add function description.

 

   @param VOID            EDES_TODO: Add parameter description

 

@@ -645,7 +645,7 @@ BOpt_GetLegacyOptions(VOID);
 // For cleaning up driver option menu

 //

 /**

-  EDES_TODO: Add function description

+  EDES_TODO: Add function description.

 

   @param VOID            EDES_TODO: Add parameter description

 

@@ -659,7 +659,7 @@ BOpt_FreeLegacyOptions(VOID);
 // this function is used to take place of all other free menu actions

 //

 /**

-  EDES_TODO: Add function description

+  EDES_TODO: Add function description.

 

   @param FreeMenu        EDES_TODO: Add parameter description

 

@@ -676,7 +676,7 @@ BOpt_FreeMenu (
 // Following are the helper functions used

 //

 /**

-  EDES_TODO: Add function description

+  EDES_TODO: Add function description.

 

   @param Str1            EDES_TODO: Add parameter description

   @param Str2            EDES_TODO: Add parameter description

@@ -691,7 +691,7 @@ BOpt_AppendFileName (
   );

 

 /**

-  EDES_TODO: Add function description

+  EDES_TODO: Add function description.

 

   @param FileName        EDES_TODO: Add parameter description

 

@@ -704,7 +704,7 @@ BOpt_IsEfiImageName (
   );

 

 /**

-  EDES_TODO: Add function description

+  EDES_TODO: Add function description.

 

   @param Dir             EDES_TODO: Add parameter description

   @param FileName        EDES_TODO: Add parameter description

@@ -722,7 +722,7 @@ BOpt_IsEfiApp (
 // Get current unused boot option number

 //

 /**

-  EDES_TODO: Add function description

+  EDES_TODO: Add function description.

 

   @param VOID            EDES_TODO: Add parameter description

 

@@ -736,7 +736,7 @@ BOpt_GetBootOptionNumber (VOID);
 // Get current unused driver option number

 //

 /**

-  EDES_TODO: Add function description

+  EDES_TODO: Add function description.

 

   @param VOID            EDES_TODO: Add parameter description

 

@@ -747,7 +747,7 @@ UINT16
 BOpt_GetDriverOptionNumber (VOID);

 

 /**

-  EDES_TODO: Add function description

+  EDES_TODO: Add function description.

 

   @param MenuType        EDES_TODO: Add parameter description

 

@@ -760,7 +760,7 @@ BOpt_CreateMenuEntry (
   );

 

 /**

-  EDES_TODO: Add function description

+  EDES_TODO: Add function description.

 

   @param MenuEntry       EDES_TODO: Add parameter description

 

@@ -773,7 +773,7 @@ BOpt_DestroyMenuEntry (
   );

 

 /**

-  EDES_TODO: Add function description

+  EDES_TODO: Add function description.

 

   @param MenuOption      EDES_TODO: Add parameter description

   @param MenuNumber      EDES_TODO: Add parameter description

@@ -791,7 +791,7 @@ BOpt_GetMenuEntry (
 // a helper function used to free pool type memory

 //

 /**

-  EDES_TODO: Add function description

+  EDES_TODO: Add function description.

 

   @param Buffer          EDES_TODO: Add parameter description

 

@@ -807,7 +807,7 @@ SafeFreePool (
 // Locate all serial io devices for console

 //

 /**

-  EDES_TODO: Add function description

+  EDES_TODO: Add function description.

 

   @param VOID            EDES_TODO: Add parameter description

 

@@ -821,7 +821,7 @@ LocateSerialIo (VOID);
 // Initializing Console menu

 //

 /**

-  EDES_TODO: Add function description

+  EDES_TODO: Add function description.

 

   @param VOID            EDES_TODO: Add parameter description

 

@@ -835,7 +835,7 @@ GetAllConsoles(VOID);
 // Get current mode information

 //

 /**

-  EDES_TODO: Add function description

+  EDES_TODO: Add function description.

 

   @param CallbackData    EDES_TODO: Add parameter description

 

@@ -851,7 +851,7 @@ GetConsoleOutMode (
 // Cleaning up console menu

 //

 /**

-  EDES_TODO: Add function description

+  EDES_TODO: Add function description.

 

   @param VOID            EDES_TODO: Add parameter description

 

@@ -862,7 +862,7 @@ EFI_STATUS
 FreeAllConsoles(VOID);

 

 /**

-  EDES_TODO: Add function description

+  EDES_TODO: Add function description.

 

   @param DevicePath      EDES_TODO: Add parameter description

 

@@ -875,7 +875,7 @@ ChangeVariableDevicePath (
 );

 

 /**

-  EDES_TODO: Add function description

+  EDES_TODO: Add function description.

 

   @param DevicePath      EDES_TODO: Add parameter description

   @param ChangeTerminal  EDES_TODO: Add parameter description

@@ -892,7 +892,7 @@ ChangeTerminalDevicePath (
 // Variable operation by menu selection

 //

 /**

-  EDES_TODO: Add function description

+  EDES_TODO: Add function description.

 

   @param CallbackData    EDES_TODO: Add parameter description

   @param NvRamMap        EDES_TODO: Add parameter description

@@ -907,7 +907,7 @@ Var_UpdateBootOption (
   );

 

 /**

-  EDES_TODO: Add function description

+  EDES_TODO: Add function description.

 

   @param VOID            EDES_TODO: Add parameter description

 

@@ -918,7 +918,7 @@ EFI_STATUS
 Var_DelBootOption (VOID);

 

 /**

-  EDES_TODO: Add function description

+  EDES_TODO: Add function description.

 

   @param VOID            EDES_TODO: Add parameter description

 

@@ -929,7 +929,7 @@ EFI_STATUS
 Var_ChangeBootOrder (VOID);

 

 /**

-  EDES_TODO: Add function description

+  EDES_TODO: Add function description.

 

   @param CallbackData    EDES_TODO: Add parameter description

   @param HiiHandle       EDES_TODO: Add parameter description

@@ -950,7 +950,7 @@ Var_UpdateDriverOption (
   );

 

 /**

-  EDES_TODO: Add function description

+  EDES_TODO: Add function description.

 

   @param VOID            EDES_TODO: Add parameter description

 

@@ -961,7 +961,7 @@ EFI_STATUS
 Var_DelDriverOption (VOID);

 

 /**

-  EDES_TODO: Add function description

+  EDES_TODO: Add function description.

 

   @param VOID            EDES_TODO: Add parameter description

 

@@ -972,7 +972,7 @@ EFI_STATUS
 Var_ChangeDriverOrder (VOID);

 

 /**

-  EDES_TODO: Add function description

+  EDES_TODO: Add function description.

 

   @param VOID            EDES_TODO: Add parameter description

 

@@ -983,7 +983,7 @@ EFI_STATUS
 Var_UpdateConsoleInpOption (VOID);

 

 /**

-  EDES_TODO: Add function description

+  EDES_TODO: Add function description.

 

   @param VOID            EDES_TODO: Add parameter description

 

@@ -994,7 +994,7 @@ EFI_STATUS
 Var_UpdateConsoleOutOption (VOID);

 

 /**

-  EDES_TODO: Add function description

+  EDES_TODO: Add function description.

 

   @param VOID            EDES_TODO: Add parameter description

 

@@ -1005,7 +1005,7 @@ EFI_STATUS
 Var_UpdateErrorOutOption (VOID);

 

 /**

-  EDES_TODO: Add function description

+  EDES_TODO: Add function description.

 

   @param VOID            EDES_TODO: Add parameter description

 

@@ -1016,7 +1016,7 @@ VOID
 Var_UpdateAllConsoleOption (VOID);

 

 /**

-  EDES_TODO: Add function description

+  EDES_TODO: Add function description.

 

   @param CallbackData    EDES_TODO: Add parameter description

 

@@ -1029,7 +1029,7 @@ Var_UpdateBootNext (
   );

 

 /**

-  EDES_TODO: Add function description

+  EDES_TODO: Add function description.

 

   @param CallbackData    EDES_TODO: Add parameter description

 

@@ -1042,7 +1042,7 @@ Var_UpdateBootOrder (
   );

 

 /**

-  EDES_TODO: Add function description

+  EDES_TODO: Add function description.

 

   @param CallbackData    EDES_TODO: Add parameter description

 

@@ -1055,7 +1055,7 @@ Var_UpdateDriverOrder (
   );

 

 /**

-  EDES_TODO: Add function description

+  EDES_TODO: Add function description.

 

   @param CallbackData    EDES_TODO: Add parameter description

 

@@ -1068,7 +1068,7 @@ Var_UpdateBBSOption (
   );

 

 /**

-  EDES_TODO: Add function description

+  EDES_TODO: Add function description.

 

   @param CallbackData    EDES_TODO: Add parameter description

 

@@ -1084,7 +1084,7 @@ Var_UpdateConMode (
 // Following are page create and refresh functions

 //

 /**

-  EDES_TODO: Add function description

+  EDES_TODO: Add function description.

 

   @param VOID            EDES_TODO: Add parameter description

 

@@ -1097,7 +1097,7 @@ RefreshUpdateData (
   );

 

 /**

-  EDES_TODO: Add function description

+  EDES_TODO: Add function description.

 

   @param LabelId         EDES_TODO: Add parameter description

   @param CallbackData    EDES_TODO: Add parameter description

@@ -1112,7 +1112,7 @@ CleanUpPage (
   );

 

 /**

-  EDES_TODO: Add function description

+  EDES_TODO: Add function description.

 

   @param CallbackData    EDES_TODO: Add parameter description

   @param UpdatingMenu    EDES_TODO: Add parameter description

@@ -1143,7 +1143,7 @@ UpdatePage (
   );

 

 /**

-  EDES_TODO: Add function description

+  EDES_TODO: Add function description.

 

   @param CallbackData    EDES_TODO: Add parameter description

 

@@ -1156,7 +1156,7 @@ UpdateBootAddPage (
   );

 

 /**

-  EDES_TODO: Add function description

+  EDES_TODO: Add function description.

 

   @param CallbackData    EDES_TODO: Add parameter description

 

@@ -1169,7 +1169,7 @@ UpdateBootDelPage (
   );

 

 /**

-  EDES_TODO: Add function description

+  EDES_TODO: Add function description.

 

   @param CallbackData    EDES_TODO: Add parameter description

 

@@ -1182,7 +1182,7 @@ UpdateDrvAddFilePage (
   );

 

 /**

-  EDES_TODO: Add function description

+  EDES_TODO: Add function description.

 

   @param CallbackData    EDES_TODO: Add parameter description

 

@@ -1195,7 +1195,7 @@ UpdateDrvAddHandlePage (
   );

 

 /**

-  EDES_TODO: Add function description

+  EDES_TODO: Add function description.

 

   @param CallbackData    EDES_TODO: Add parameter description

 

@@ -1208,7 +1208,7 @@ UpdateDrvDelPage (
   );

 

 /**

-  EDES_TODO: Add function description

+  EDES_TODO: Add function description.

 

   @param CallbackData    EDES_TODO: Add parameter description

 

@@ -1221,7 +1221,7 @@ UpdateDriverAddHandleDescPage (
   );

 

 /**

-  EDES_TODO: Add function description

+  EDES_TODO: Add function description.

 

   @param CallbackData    EDES_TODO: Add parameter description

 

@@ -1234,7 +1234,7 @@ UpdateBootTimeOut (
   );

 

 /**

-  EDES_TODO: Add function description

+  EDES_TODO: Add function description.

 

   @param CallbackData    EDES_TODO: Add parameter description

 

@@ -1247,7 +1247,7 @@ UpdateConInPage (
   );

 

 /**

-  EDES_TODO: Add function description

+  EDES_TODO: Add function description.

 

   @param CallbackData    EDES_TODO: Add parameter description

 

@@ -1260,7 +1260,7 @@ UpdateConOutPage (
   );

 

 /**

-  EDES_TODO: Add function description

+  EDES_TODO: Add function description.

 

   @param CallbackData    EDES_TODO: Add parameter description

 

@@ -1273,7 +1273,7 @@ UpdateStdErrPage (
   );

 

 /**

-  EDES_TODO: Add function description

+  EDES_TODO: Add function description.

 

   @param UpdatePageId    EDES_TODO: Add parameter description

   @param CallbackData    EDES_TODO: Add parameter description

@@ -1288,7 +1288,7 @@ UpdatePageBody (
   );

 

 /**

-  EDES_TODO: Add function description

+  EDES_TODO: Add function description.

 

   @param CallbackData    EDES_TODO: Add parameter description

 

@@ -1301,7 +1301,7 @@ UpdateCOM1Page (
   );

 

 /**

-  EDES_TODO: Add function description

+  EDES_TODO: Add function description.

 

   @param CallbackData    EDES_TODO: Add parameter description

 

@@ -1314,7 +1314,7 @@ UpdateCOM2Page (
   );

 

 /**

-  EDES_TODO: Add function description

+  EDES_TODO: Add function description.

 

   @param CallbackData    EDES_TODO: Add parameter description

 

@@ -1327,7 +1327,7 @@ UpdateBootOrderPage (
   );

 

 /**

-  EDES_TODO: Add function description

+  EDES_TODO: Add function description.

 

   @param CallbackData    EDES_TODO: Add parameter description

 

@@ -1340,7 +1340,7 @@ UpdateDriverOrderPage (
   );

 

 /**

-  EDES_TODO: Add function description

+  EDES_TODO: Add function description.

 

   @param CallbackData    EDES_TODO: Add parameter description

 

@@ -1353,7 +1353,7 @@ UpdateBootNextPage (
   );

 

 /**

-  EDES_TODO: Add function description

+  EDES_TODO: Add function description.

 

   @param CallbackData    EDES_TODO: Add parameter description

 

@@ -1366,7 +1366,7 @@ UpdateTimeOutPage (
   );

 

 /**

-  EDES_TODO: Add function description

+  EDES_TODO: Add function description.

 

   @param CallbackData    EDES_TODO: Add parameter description

 

@@ -1379,7 +1379,7 @@ UpdateTerminalPage (
   );

 

 /**

-  EDES_TODO: Add function description

+  EDES_TODO: Add function description.

 

   @param CallbackData    EDES_TODO: Add parameter description

 

@@ -1392,7 +1392,7 @@ UpdateConModePage (
   );

 

 /**

-  EDES_TODO: Add function description

+  EDES_TODO: Add function description.

 

   @param CallbackData    EDES_TODO: Add parameter description

 

@@ -1405,7 +1405,7 @@ UpdateConCOMPage (
   );

 

 /**

-  EDES_TODO: Add function description

+  EDES_TODO: Add function description.

 

   @param UpdatePageId    EDES_TODO: Add parameter description

   @param CallbackData    EDES_TODO: Add parameter description

@@ -1420,7 +1420,7 @@ UpdateSetLegacyDeviceOrderPage (
 );

 

 /**

-  EDES_TODO: Add function description

+  EDES_TODO: Add function description.

 

   @param BbsType         EDES_TODO: Add parameter description

   @param BbsFlag         EDES_TODO: Add parameter description

@@ -1435,7 +1435,7 @@ BootLegacy (
 );

 

 /**

-  EDES_TODO: Add function description

+  EDES_TODO: Add function description.

 

   @param TerminalNumber  EDES_TODO: Add parameter description

 

@@ -1448,7 +1448,7 @@ GetCurrentTerminal (
 );

 

 /**

-  EDES_TODO: Add function description

+  EDES_TODO: Add function description.

 

   @param DeviceHandle    EDES_TODO: Add parameter description

 

@@ -1461,7 +1461,7 @@ EfiLibOpenRoot (
   );

 

 /**

-  EDES_TODO: Add function description

+  EDES_TODO: Add function description.

 

   @param FHand           EDES_TODO: Add parameter description

 

@@ -1474,7 +1474,7 @@ EfiLibFileSystemVolumeLabelInfo (
   );

 

 /**

-  EDES_TODO: Add function description

+  EDES_TODO: Add function description.

 

   @param FHand           EDES_TODO: Add parameter description

 

@@ -1487,7 +1487,7 @@ EfiLibFileInfo (
   );

 

 /**

-  EDES_TODO: Add function description

+  EDES_TODO: Add function description.

 

   @param UStr            EDES_TODO: Add parameter description

   @param Length          EDES_TODO: Add parameter description

@@ -1504,7 +1504,7 @@ UnicodeToAscii (
   );

 

 /**

-  EDES_TODO: Add function description

+  EDES_TODO: Add function description.

 

   @param DevPath         EDES_TODO: Add parameter description

 

@@ -1517,7 +1517,7 @@ DevicePathToStr (
   );

 

 /**

-  EDES_TODO: Add function description

+  EDES_TODO: Add function description.

 

   @param Size            EDES_TODO: Add parameter description

 

@@ -1530,7 +1530,7 @@ EfiAllocateZeroPool (
   );

 

 /**

-  EDES_TODO: Add function description

+  EDES_TODO: Add function description.

 

   @param ProtocolGuid    EDES_TODO: Add parameter description

   @param Interface       EDES_TODO: Add parameter description

@@ -1545,7 +1545,7 @@ EfiLibLocateProtocol (
   );

 

 /**

-  EDES_TODO: Add function description

+  EDES_TODO: Add function description.

 

   @param OldPool         EDES_TODO: Add parameter description

   @param OldSize         EDES_TODO: Add parameter description

@@ -1562,7 +1562,7 @@ EfiReallocatePool (
   );

 

 /**

-  EDES_TODO: Add function description

+  EDES_TODO: Add function description.

 

   @param DevPath         EDES_TODO: Add parameter description

 

@@ -1575,7 +1575,7 @@ DevicePathToStr (
   );

 

 /**

-  EDES_TODO: Add function description

+  EDES_TODO: Add function description.

 

   @param Name            EDES_TODO: Add parameter description

   @param VendorGuid      EDES_TODO: Add parameter description

@@ -1592,7 +1592,7 @@ BdsLibGetVariableAndSize (
   );

 

 /**

-  EDES_TODO: Add function description

+  EDES_TODO: Add function description.

 

   @param VarName         EDES_TODO: Add parameter description

   @param VarGuid         EDES_TODO: Add parameter description

@@ -1607,7 +1607,7 @@ EfiLibDeleteVariable (
   );

 

 /**

-  EDES_TODO: Add function description

+  EDES_TODO: Add function description.

 

   @param Src             EDES_TODO: Add parameter description

 

@@ -1620,7 +1620,7 @@ EfiStrDuplicate (
   );

 

 /**

-  EDES_TODO: Add function description

+  EDES_TODO: Add function description.

 

   @param Multi           EDES_TODO: Add parameter description

   @param Single          EDES_TODO: Add parameter description

@@ -1635,7 +1635,7 @@ EfiLibMatchDevicePaths (
   );

 

 /**

-  EDES_TODO: Add function description

+  EDES_TODO: Add function description.

 

   @param DevicePath      EDES_TODO: Add parameter description

 

@@ -1648,7 +1648,7 @@ EfiDevicePathInstanceCount (
   );

 

 /**

-  EDES_TODO: Add function description

+  EDES_TODO: Add function description.

 

   @param CallbackData    EDES_TODO: Add parameter description

   @param HiiHandle       EDES_TODO: Add parameter description

@@ -1665,7 +1665,7 @@ CreateMenuStringToken (
   );

 

 /**

-  EDES_TODO: Add function description

+  EDES_TODO: Add function description.

 

   @param DevPath         EDES_TODO: Add parameter description

 

@@ -1678,7 +1678,7 @@ EfiLibStrFromDatahub (
   );

 

 /**

-  EDES_TODO: Add function description

+  EDES_TODO: Add function description.

 

   @param DeviceType      EDES_TODO: Add parameter description

   @param OptionIndex     EDES_TODO: Add parameter description

@@ -1695,7 +1695,7 @@ GetLegacyBootOptionVar (
  );

 

 /**

-  EDES_TODO: Add function description

+  EDES_TODO: Add function description.

 

   @param VOID            EDES_TODO: Add parameter description

 

@@ -1708,7 +1708,7 @@ InitializeBM (
   );

 

 /**

-  EDES_TODO: Add function description

+  EDES_TODO: Add function description.

 

   @param VOID            EDES_TODO: Add parameter description

 

@@ -1721,7 +1721,7 @@ BdsStartBootMaint (
   );

 

 /**

-  EDES_TODO: Add function description

+  EDES_TODO: Add function description.

 

   @param VOID            EDES_TODO: Add parameter description

 

@@ -1732,7 +1732,7 @@ VOID
 InitializeStringDepository (VOID);

 

 /**

-  EDES_TODO: Add function description

+  EDES_TODO: Add function description.

 

   @param CallbackData    EDES_TODO: Add parameter description

   @param StringDepository EDES_TODO: Add parameter description

@@ -1747,7 +1747,7 @@ GetStringTokenFromDepository (
   );

 

 /**

-  EDES_TODO: Add function description

+  EDES_TODO: Add function description.

 

   @param VOID            EDES_TODO: Add parameter description

 

@@ -1760,7 +1760,7 @@ ReclaimStringDepository (
   );

 

 /**

-  EDES_TODO: Add function description

+  EDES_TODO: Add function description.

 

   @param VOID            EDES_TODO: Add parameter description

 

@@ -1773,7 +1773,7 @@ CleanUpStringDepository (
   );

 

 /**

-  EDES_TODO: Add function description

+  EDES_TODO: Add function description.

 

   @param Private         EDES_TODO: Add parameter description

   @param CurrentFakeNVMap EDES_TODO: Add parameter description

@@ -1790,7 +1790,7 @@ ApplyChangeHandler (
   );

 

 /**

-  EDES_TODO: Add function description

+  EDES_TODO: Add function description.

 

   @param Private         EDES_TODO: Add parameter description

   @param CurrentFakeNVMap EDES_TODO: Add parameter description

@@ -1805,7 +1805,7 @@ DiscardChangeHandler (
   );

 

 /**

-  EDES_TODO: Add function description

+  EDES_TODO: Add function description.

 

   @param Private         EDES_TODO: Add parameter description

   @param NewPageId       EDES_TODO: Add parameter description

@@ -1820,7 +1820,7 @@ UpdatePageId (
   );

 

 /**

-  EDES_TODO: Add function description

+  EDES_TODO: Add function description.

 

   @param FileContext     EDES_TODO: Add parameter description

 

@@ -1833,7 +1833,7 @@ BootThisFile (
   );

 

 /**

-  EDES_TODO: Add function description

+  EDES_TODO: Add function description.

 

   @param CallbackData    EDES_TODO: Add parameter description

   @param KeyValue        EDES_TODO: Add parameter description

@@ -1848,7 +1848,7 @@ UpdateFileExplorer (
   );

 

 /**

-  EDES_TODO: Add function description

+  EDES_TODO: Add function description.

 

   @param This            EDES_TODO: Add parameter description

   @param Action          EDES_TODO: Add parameter description

@@ -1872,7 +1872,7 @@ FileExplorerCallback (
   );

 

 /**

-  EDES_TODO: Add function description

+  EDES_TODO: Add function description.

 

   @param CallbackData    EDES_TODO: Add parameter description

 

@@ -1885,7 +1885,7 @@ FormSetDispatcher (
   );

 

 /**

-  EDES_TODO: Add function description

+  EDES_TODO: Add function description.

 

   @param Name            EDES_TODO: Add parameter description

   @param VendorGuid      EDES_TODO: Add parameter description

diff --git a/MdeModulePkg/Universal/BdsDxe/BootMaint/BootOption.c b/MdeModulePkg/Universal/BdsDxe/BootMaint/BootOption.c
index ebaa48508e..89c8612d46 100644
--- a/MdeModulePkg/Universal/BdsDxe/BootMaint/BootOption.c
+++ b/MdeModulePkg/Universal/BdsDxe/BootMaint/BootOption.c
@@ -20,7 +20,7 @@ WITHOUT WARRANTIES OR REPRESENTATIONS OF ANY KIND, EITHER EXPRESS OR IMPLIED.
 #include "BBSsupport.h"

 

 /**

-  EDES_TODO: Add function description

+  EDES_TODO: Add function description.

 

   @param MenuType        Use this parameter to identify current

                          Menu type

@@ -92,7 +92,7 @@ BOpt_CreateMenuEntry (
 }

 

 /**

-  EDES_TODO: Add function description

+  EDES_TODO: Add function description.

 

   @param MenuEntry       EDES_TODO: Add parameter description

 

@@ -173,7 +173,7 @@ BOpt_DestroyMenuEntry (
 }

 

 /**

-  EDES_TODO: Add function description

+  EDES_TODO: Add function description.

 

   @param MenuOption      EDES_TODO: Add parameter description

   @param MenuNumber      EDES_TODO: Add parameter description

@@ -206,7 +206,7 @@ BOpt_GetMenuEntry (
 }

 

 /**

-  EDES_TODO: Add function description

+  EDES_TODO: Add function description.

 

   @param CallbackData    BMM context data

 

@@ -338,7 +338,7 @@ BOpt_FindFileSystem (
       FileContext->Handle     = SimpleFsHandle[Index];

       MenuEntry->OptionNumber = Index;

       FileContext->FHandle    = EfiLibOpenRoot (FileContext->Handle);

-      if (!FileContext->FHandle) {

+      if (FileContext->FHandle == NULL) {

         BOpt_DestroyMenuEntry (MenuEntry);

         continue;

       }

@@ -583,7 +583,7 @@ Returns:
   }

 

   DirInfo = EfiLibFileInfo (NewDir);

-  if (!DirInfo) {

+  if (DirInfo == NULL) {

     return EFI_NOT_FOUND;

   }

 

@@ -598,7 +598,7 @@ Returns:
 

   DirBufferSize = sizeof (EFI_FILE_INFO) + 1024;

   DirInfo       = EfiAllocateZeroPool (DirBufferSize);

-  if (!DirInfo) {

+  if (DirInfo == NULL) {

     return EFI_OUT_OF_RESOURCES;

   }

   //

@@ -908,7 +908,7 @@ Returns:
               &BootNextSize

               );

 

-  if (BootNext) {

+  if (BootNext != NULL) {

     if (BootNextSize != sizeof (UINT16)) {

       SafeFreePool (BootNext);

       BootNext = NULL;

@@ -925,19 +925,19 @@ Returns:
                           &gEfiGlobalVariableGuid,

                           &BootOptionSize

                           );

-    if (!LoadOptionFromVar) {

+    if (LoadOptionFromVar == NULL) {

       continue;

     }

 

     LoadOption = EfiAllocateZeroPool (BootOptionSize);

-    if (!LoadOption) {

+    if (LoadOption == NULL) {

       continue;

     }

 

     CopyMem (LoadOption, LoadOptionFromVar, BootOptionSize);

     SafeFreePool (LoadOptionFromVar);

 

-    if (BootNext) {

+    if (BootNext != NULL) {

       BootNextFlag = (BOOLEAN) (*BootNext == BootOrderList[Index]);

     } else {

       BootNextFlag = FALSE;

@@ -1086,7 +1086,7 @@ BdsStrCpy (
   ASSERT (Destination != NULL);

 

   ReturnValue = Destination;

-  while (*Source) {

+  while (*Source != L'\0') {

     *(Destination++) = *(Source++);

   }

   *Destination = 0;

@@ -1180,7 +1180,7 @@ Returns:
   //

   // Search for ".efi" extension

   //

-  while (*FileName) {

+  while (*FileName != L'\0') {

     if (FileName[0] == '.') {

       if (FileName[1] == 'e' || FileName[1] == 'E') {

         if (FileName[2] == 'f' || FileName[2] == 'F') {

@@ -1406,7 +1406,7 @@ Returns:
                     &gEfiGlobalVariableGuid,

                     &BootOrderListSize

                     );

-  if (BootOrderList) {

+  if (BootOrderList != NULL) {

     //

     // already have Boot####

     //

@@ -1480,7 +1480,7 @@ Returns:
                       &gEfiGlobalVariableGuid,

                       &DriverOrderListSize

                       );

-  if (DriverOrderList) {

+  if (DriverOrderList != NULL) {

     //

     // already have Driver####

     //

@@ -1575,12 +1575,12 @@ Returns:
                           &gEfiGlobalVariableGuid,

                           &DriverOptionSize

                           );

-    if (!LoadOptionFromVar) {

+    if (LoadOptionFromVar == NULL) {

       continue;

     }

 

     LoadOption = EfiAllocateZeroPool (DriverOptionSize);

-    if (!LoadOption) {

+    if (LoadOption == NULL) {

       continue;

     }

 

diff --git a/MdeModulePkg/Universal/BdsDxe/BootMaint/ConsoleOption.c b/MdeModulePkg/Universal/BdsDxe/BootMaint/ConsoleOption.c
index e3a22f26d0..270948633b 100644
--- a/MdeModulePkg/Universal/BdsDxe/BootMaint/ConsoleOption.c
+++ b/MdeModulePkg/Universal/BdsDxe/BootMaint/ConsoleOption.c
@@ -15,7 +15,7 @@ WITHOUT WARRANTIES OR REPRESENTATIONS OF ANY KIND, EITHER EXPRESS OR IMPLIED.
 #include "BootMaint.h"

 

 /**

-  EDES_TODO: Add function description

+  EDES_TODO: Add function description.

 

   @param DevPath         EDES_TODO: Add parameter description

 

@@ -28,7 +28,7 @@ DevicePathInstanceDup (
   );

 

 /**

-  EDES_TODO: Add function description

+  EDES_TODO: Add function description.

 

   @param DevicePath      EDES_TODO: Add parameter description

 

@@ -41,7 +41,7 @@ UpdateComAttributeFromVariable (
   );

 

 /**

-  EDES_TODO: Add function description

+  EDES_TODO: Add function description.

 

   @param DevicePath      EDES_TODO: Add parameter description

   @param ChangeTerminal  EDES_TODO: Add parameter description

@@ -162,7 +162,7 @@ ChangeTerminalDevicePath (
 }

 

 /**

-  EDES_TODO: Add function description

+  EDES_TODO: Add function description.

 

   @param DevicePath      EDES_TODO: Add parameter description

 

@@ -328,7 +328,7 @@ SortedUartHandle (
 }

 

 /**

-  EDES_TODO: Add function description

+  EDES_TODO: Add function description.

 

   @param DevicePath      EDES_TODO: Add parameter description

   @param Termi           EDES_TODO: Add parameter description

@@ -420,7 +420,7 @@ LocateSerialIo (
 

     if (CompareMem (&Acpi->HID, &Match, sizeof (UINT32)) == 0) {

       NewMenuEntry = BOpt_CreateMenuEntry (BM_TERMINAL_CONTEXT_SELECT);

-      if (!NewMenuEntry) {

+      if (NewMenuEntry == NULL) {

         SafeFreePool (Handles);

         return EFI_OUT_OF_RESOURCES;

       }

@@ -482,15 +482,15 @@ LocateSerialIo (
   OutDevicePath = EfiLibGetVariable (L"ConOut", &gEfiGlobalVariableGuid);

   InpDevicePath = EfiLibGetVariable (L"ConIn", &gEfiGlobalVariableGuid);

   ErrDevicePath = EfiLibGetVariable (L"ErrOut", &gEfiGlobalVariableGuid);

-  if (OutDevicePath) {

+  if (OutDevicePath != NULL) {

     UpdateComAttributeFromVariable (OutDevicePath);

   }

 

-  if (InpDevicePath) {

+  if (InpDevicePath != NULL) {

     UpdateComAttributeFromVariable (InpDevicePath);

   }

 

-  if (ErrDevicePath) {

+  if (ErrDevicePath != NULL) {

     UpdateComAttributeFromVariable (ErrDevicePath);

   }

 

@@ -697,12 +697,12 @@ DevicePathInstanceDup (
   // Make a copy and set proper end type

   //

   NewDevPath = NULL;

-  if (Size) {

+  if (Size != 0) {

     NewDevPath = EfiAllocateZeroPool (Size);

     ASSERT (NewDevPath != NULL);

   }

 

-  if (NewDevPath) {

+  if (NewDevPath != NULL) {

     CopyMem (NewDevPath, DevicePathInst, Size);

     Ptr = (UINT8 *) NewDevPath;

     Ptr += Size - sizeof (EFI_DEVICE_PATH_PROTOCOL);

@@ -714,7 +714,7 @@ DevicePathInstanceDup (
 }

 

 /**

-  EDES_TODO: Add function description

+  EDES_TODO: Add function description.

 

   @param ConsoleMenuType EDES_TODO: Add parameter description

 

diff --git a/MdeModulePkg/Universal/BdsDxe/BootMaint/UpdatePage.c b/MdeModulePkg/Universal/BdsDxe/BootMaint/UpdatePage.c
index be84fdb563..3dd5df7997 100644
--- a/MdeModulePkg/Universal/BdsDxe/BootMaint/UpdatePage.c
+++ b/MdeModulePkg/Universal/BdsDxe/BootMaint/UpdatePage.c
@@ -32,7 +32,7 @@ RefreshUpdateData (
 }

 

 /**

-  EDES_TODO: Add function description

+  EDES_TODO: Add function description.

 

   @param CallbackData    EDES_TODO: Add parameter description

 

@@ -64,7 +64,7 @@ UpdatePageStart (
 }

 

 /**

-  EDES_TODO: Add function description

+  EDES_TODO: Add function description.

 

   @param CallbackData    EDES_TODO: Add parameter description

 

@@ -121,7 +121,7 @@ UpdatePageEnd (
 }

 

 /**

-  EDES_TODO: Add function description

+  EDES_TODO: Add function description.

 

   @param LabelId         EDES_TODO: Add parameter description

   @param CallbackData    EDES_TODO: Add parameter description

@@ -151,7 +151,7 @@ CleanUpPage (
 }

 

 /**

-  EDES_TODO: Add function description

+  EDES_TODO: Add function description.

 

   @param FileContext     EDES_TODO: Add parameter description

 

@@ -192,7 +192,7 @@ BootThisFile (
 }

 

 /**

-  EDES_TODO: Add function description

+  EDES_TODO: Add function description.

 

   @param CallbackData    EDES_TODO: Add parameter description

 

@@ -229,7 +229,7 @@ UpdateConCOMPage (
 }

 

 /**

-  EDES_TODO: Add function description

+  EDES_TODO: Add function description.

 

   @param CallbackData    EDES_TODO: Add parameter description

 

@@ -276,7 +276,7 @@ UpdateBootDelPage (
 }

 

 /**

-  EDES_TODO: Add function description

+  EDES_TODO: Add function description.

 

   @param CallbackData    EDES_TODO: Add parameter description

 

@@ -312,7 +312,7 @@ UpdateDrvAddHandlePage (
 }

 

 /**

-  EDES_TODO: Add function description

+  EDES_TODO: Add function description.

 

   @param CallbackData    EDES_TODO: Add parameter description

 

@@ -357,7 +357,7 @@ UpdateDrvDelPage (
 }

 

 /**

-  EDES_TODO: Add function description

+  EDES_TODO: Add function description.

 

   @param CallbackData    EDES_TODO: Add parameter description

 

@@ -427,7 +427,7 @@ UpdateDriverAddHandleDescPage (
 }

 

 /**

-  EDES_TODO: Add function description

+  EDES_TODO: Add function description.

 

   @param UpdatePageId    EDES_TODO: Add parameter description

   @param ConsoleMenu     EDES_TODO: Add parameter description

@@ -482,9 +482,9 @@ UpdateConsolePage (
     NewMenuEntry        = BOpt_GetMenuEntry (&TerminalMenu, Index2);

     NewTerminalContext  = (BM_TERMINAL_CONTEXT *) NewMenuEntry->VariableContext;

 

-    if ((NewTerminalContext->IsConIn && (UpdatePageId == FORM_CON_IN_ID)) ||

-        (NewTerminalContext->IsConOut && (UpdatePageId == FORM_CON_OUT_ID)) ||

-        (NewTerminalContext->IsStdErr && (UpdatePageId == FORM_CON_ERR_ID))

+    if (((NewTerminalContext->IsConIn != 0) && (UpdatePageId == FORM_CON_IN_ID)) ||

+        ((NewTerminalContext->IsConOut != 0) && (UpdatePageId == FORM_CON_OUT_ID)) ||

+        ((NewTerminalContext->IsStdErr != 0) && (UpdatePageId == FORM_CON_ERR_ID))

         ) {

       CheckFlags |= EFI_IFR_CHECKBOX_DEFAULT;

       CallbackData->BmmFakeNvData.ConsoleCheck[Index] = TRUE;

@@ -510,7 +510,7 @@ UpdateConsolePage (
 }

 

 /**

-  EDES_TODO: Add function description

+  EDES_TODO: Add function description.

 

   @param UpdatePageId    EDES_TODO: Add parameter description

   @param OptionMenu      EDES_TODO: Add parameter description

@@ -580,7 +580,7 @@ UpdateOrderPage (
 }

 

 /**

-  EDES_TODO: Add function description

+  EDES_TODO: Add function description.

 

   @param CallbackData    EDES_TODO: Add parameter description

 

@@ -653,7 +653,7 @@ UpdateBootNextPage (
 }

 

 /**

-  EDES_TODO: Add function description

+  EDES_TODO: Add function description.

 

   @param CallbackData    EDES_TODO: Add parameter description

 

@@ -802,7 +802,7 @@ UpdateConModePage (
 }

 

 /**

-  EDES_TODO: Add function description

+  EDES_TODO: Add function description.

 

   @param CallbackData    EDES_TODO: Add parameter description

 

@@ -976,7 +976,7 @@ UpdateTerminalPage (
 }

 

 /**

-  EDES_TODO: Add function description

+  EDES_TODO: Add function description.

 

   @param UpdatePageId    EDES_TODO: Add parameter description

   @param CallbackData    EDES_TODO: Add parameter description

@@ -1018,7 +1018,7 @@ UpdatePageBody (
 }

 

 /**

-  EDES_TODO: Add function description

+  EDES_TODO: Add function description.

 

   @param DeviceType      EDES_TODO: Add parameter description

   @param OptionIndex     EDES_TODO: Add parameter description

@@ -1100,7 +1100,7 @@ GetLegacyBootOptionVar (
 }

 

 /**

-  EDES_TODO: Add function description

+  EDES_TODO: Add function description.

 

   @param UpdatePageId    EDES_TODO: Add parameter description

   @param CallbackData    EDES_TODO: Add parameter description

@@ -1304,7 +1304,7 @@ UpdateSetLegacyDeviceOrderPage (
 }

 

 /**

-  EDES_TODO: Add function description

+  EDES_TODO: Add function description.

 

   @param Private         EDES_TODO: Add parameter description

   @param NewPageId       EDES_TODO: Add parameter description

diff --git a/MdeModulePkg/Universal/BdsDxe/BootMaint/Variable.c b/MdeModulePkg/Universal/BdsDxe/BootMaint/Variable.c
index 498a40f722..6479cd0f17 100644
--- a/MdeModulePkg/Universal/BdsDxe/BootMaint/Variable.c
+++ b/MdeModulePkg/Universal/BdsDxe/BootMaint/Variable.c
@@ -121,7 +121,7 @@ Var_ChangeBootOrder (
   //

   // If exists, delete it to hold new BootOrder

   //

-  if (BootOrderList) {

+  if (BootOrderList != NULL) {

     EfiLibDeleteVariable (L"BootOrder", &gEfiGlobalVariableGuid);

     SafeFreePool (BootOrderList);

     BootOrderList = NULL;

@@ -271,7 +271,7 @@ Var_ChangeDriverOrder (
   //

   // If exists, delete it to hold new DriverOrder

   //

-  if (DriverOrderList) {

+  if (DriverOrderList != NULL) {

     EfiLibDeleteVariable (L"DriverOrder", &gEfiGlobalVariableGuid);

     SafeFreePool (DriverOrderList);

     DriverOrderList = NULL;

@@ -315,7 +315,7 @@ Var_ChangeDriverOrder (
 }

 

 /**

-  EDES_TODO: Add function description

+  EDES_TODO: Add function description.

 

   @param VOID            EDES_TODO: Add parameter description

 

@@ -335,7 +335,7 @@ Var_UpdateAllConsoleOption (
   OutDevicePath = EfiLibGetVariable (L"ConOut", &gEfiGlobalVariableGuid);

   InpDevicePath = EfiLibGetVariable (L"ConIn", &gEfiGlobalVariableGuid);

   ErrDevicePath = EfiLibGetVariable (L"ErrOut", &gEfiGlobalVariableGuid);

-  if (OutDevicePath) {

+  if (OutDevicePath != NULL) {

     ChangeVariableDevicePath (OutDevicePath);

     Status = gRT->SetVariable (

                     L"ConOut",

@@ -347,7 +347,7 @@ Var_UpdateAllConsoleOption (
     ASSERT (!EFI_ERROR (Status));

   }

 

-  if (InpDevicePath) {

+  if (InpDevicePath != NULL) {

     ChangeVariableDevicePath (InpDevicePath);

     Status = gRT->SetVariable (

                     L"ConIn",

@@ -359,7 +359,7 @@ Var_UpdateAllConsoleOption (
     ASSERT (!EFI_ERROR (Status));

   }

 

-  if (ErrDevicePath) {

+  if (ErrDevicePath != NULL) {

     ChangeVariableDevicePath (ErrDevicePath);

     Status = gRT->SetVariable (

                     L"ErrOut",

@@ -373,7 +373,7 @@ Var_UpdateAllConsoleOption (
 }

 

 /**

-  EDES_TODO: Add function description

+  EDES_TODO: Add function description.

 

   @param ConsoleName     EDES_TODO: Add parameter description

   @param ConsoleMenu     EDES_TODO: Add parameter description

@@ -430,9 +430,9 @@ Var_UpdateConsoleOption (
     }

 

     NewTerminalContext = (BM_TERMINAL_CONTEXT *) NewMenuEntry->VariableContext;

-    if ((NewTerminalContext->IsConIn && (UpdatePageId == FORM_CON_IN_ID)) ||

-        (NewTerminalContext->IsConOut && (UpdatePageId == FORM_CON_OUT_ID)) ||

-        (NewTerminalContext->IsStdErr && (UpdatePageId == FORM_CON_ERR_ID))

+    if (((NewTerminalContext->IsConIn != 0) && (UpdatePageId == FORM_CON_IN_ID)) ||

+        ((NewTerminalContext->IsConOut != 0)  && (UpdatePageId == FORM_CON_OUT_ID)) ||

+        ((NewTerminalContext->IsStdErr  != 0) && (UpdatePageId == FORM_CON_ERR_ID))

         ) {

       Vendor.Header.Type    = MESSAGING_DEVICE_PATH;

       Vendor.Header.SubType = MSG_VENDOR_DP;

@@ -455,7 +455,7 @@ Var_UpdateConsoleOption (
     }

   }

 

-  if (ConDevicePath) {

+  if (ConDevicePath != NULL) {

     Status = gRT->SetVariable (

                     ConsoleName,

                     &gEfiGlobalVariableGuid,

@@ -473,7 +473,7 @@ Var_UpdateConsoleOption (
 }

 

 /**

-  EDES_TODO: Add function description

+  EDES_TODO: Add function description.

 

   @param VOID            EDES_TODO: Add parameter description

 

@@ -489,7 +489,7 @@ Var_UpdateConsoleInpOption (
 }

 

 /**

-  EDES_TODO: Add function description

+  EDES_TODO: Add function description.

 

   @param VOID            EDES_TODO: Add parameter description

 

@@ -505,7 +505,7 @@ Var_UpdateConsoleOutOption (
 }

 

 /**

-  EDES_TODO: Add function description

+  EDES_TODO: Add function description.

 

   @param VOID            EDES_TODO: Add parameter description

 

@@ -521,7 +521,7 @@ Var_UpdateErrorOutOption (
 }

 

 /**

-  EDES_TODO: Add function description

+  EDES_TODO: Add function description.

 

   @param CallbackData    EDES_TODO: Add parameter description

   @param HiiHandle       EDES_TODO: Add parameter description

@@ -698,7 +698,7 @@ Var_UpdateDriverOption (
 }

 

 /**

-  EDES_TODO: Add function description

+  EDES_TODO: Add function description.

 

   @param CallbackData    EDES_TODO: Add parameter description

   @param NvRamMap        EDES_TODO: Add parameter description

@@ -865,7 +865,7 @@ Var_UpdateBootOption (
 }

 

 /**

-  EDES_TODO: Add function description

+  EDES_TODO: Add function description.

 

   @param CallbackData    EDES_TODO: Add parameter description

 

@@ -922,7 +922,7 @@ Var_UpdateBootNext (
 }

 

 /**

-  EDES_TODO: Add function description

+  EDES_TODO: Add function description.

 

   @param CallbackData    EDES_TODO: Add parameter description

 

@@ -954,18 +954,18 @@ Var_UpdateBootOrder (
                     );

 

   NewBootOrderList = EfiAllocateZeroPool (BootOrderListSize);

-  if (!NewBootOrderList) {

+  if (NewBootOrderList == NULL) {

     return EFI_OUT_OF_RESOURCES;

   }

 

   Map = EfiAllocateZeroPool (BootOrderListSize / sizeof (UINT16));

-  if (!Map) {

+  if (Map == NULL) {

     return EFI_OUT_OF_RESOURCES;

   }

   //

   // If exists, delete it to hold new BootOrder

   //

-  if (BootOrderList) {

+  if (BootOrderList != NULL) {

     EfiLibDeleteVariable (L"BootOrder", &gEfiGlobalVariableGuid);

   }

 

@@ -995,7 +995,7 @@ Var_UpdateBootOrder (
 }

 

 /**

-  EDES_TODO: Add function description

+  EDES_TODO: Add function description.

 

   @param CallbackData    EDES_TODO: Add parameter description

 

@@ -1027,13 +1027,13 @@ Var_UpdateDriverOrder (
 

   NewDriverOrderList = EfiAllocateZeroPool (DriverOrderListSize);

 

-  if (!NewDriverOrderList) {

+  if (NewDriverOrderList == NULL) {

     return EFI_OUT_OF_RESOURCES;

   }

   //

   // If exists, delete it to hold new DriverOrder

   //

-  if (DriverOrderList) {

+  if (DriverOrderList != NULL) {

     EfiLibDeleteVariable (L"DriverOrder", &gEfiGlobalVariableGuid);

   }

 

@@ -1060,7 +1060,7 @@ Var_UpdateDriverOrder (
 }

 

 /**

-  EDES_TODO: Add function description

+  EDES_TODO: Add function description.

 

   @param CallbackData    EDES_TODO: Add parameter description

 

@@ -1202,7 +1202,7 @@ Var_UpdateBBSOption (
     Tmp &= 0xFF;

     Pos = Tmp / 8;

     Bit = 7 - (Tmp % 8);

-    if (DisMap[Pos] & (1 << Bit)) {

+    if ((DisMap[Pos] & (1 << Bit)) != 0) {

       NewOrder[Index] = (UINT16) (0xFF00 | Tmp);

       Index++;

     }

@@ -1364,7 +1364,7 @@ Var_UpdateBBSOption (
 }

 

 /**

-  EDES_TODO: Add function description

+  EDES_TODO: Add function description.

 

   @param CallbackData    EDES_TODO: Add parameter description

 

diff --git a/MdeModulePkg/Universal/BdsDxe/BootMngr/BootManager.h b/MdeModulePkg/Universal/BdsDxe/BootMngr/BootManager.h
index 5c83394fc5..3b40fa6d42 100644
--- a/MdeModulePkg/Universal/BdsDxe/BootMngr/BootManager.h
+++ b/MdeModulePkg/Universal/BdsDxe/BootMngr/BootManager.h
@@ -53,7 +53,7 @@ typedef struct {
 } BOOT_MANAGER_CALLBACK_DATA;

 

 /**

-  EDES_TODO: Add function description

+  EDES_TODO: Add function description.

 

   @param This            EDES_TODO: Add parameter description

   @param Action          EDES_TODO: Add parameter description

@@ -78,7 +78,7 @@ BootManagerCallback (
 ;

 

 /**

-  EDES_TODO: Add function description

+  EDES_TODO: Add function description.

 

   @param VOID            EDES_TODO: Add parameter description

 

@@ -92,7 +92,7 @@ InitializeBootManager (
 ;

 

 /**

-  EDES_TODO: Add function description

+  EDES_TODO: Add function description.

 

   @param VOID            EDES_TODO: Add parameter description

 

diff --git a/MdeModulePkg/Universal/BdsDxe/DeviceMngr/DeviceManager.h b/MdeModulePkg/Universal/BdsDxe/DeviceMngr/DeviceManager.h
index 3ced4c20c0..3b1564399e 100644
--- a/MdeModulePkg/Universal/BdsDxe/DeviceMngr/DeviceManager.h
+++ b/MdeModulePkg/Universal/BdsDxe/DeviceMngr/DeviceManager.h
@@ -72,7 +72,7 @@ typedef struct {
 } DEVICE_MANAGER_MENU_ITEM;

 

 /**

-  EDES_TODO: Add function description

+  EDES_TODO: Add function description.

 

   @param This            EDES_TODO: Add parameter description

   @param Action          EDES_TODO: Add parameter description

@@ -97,7 +97,7 @@ DeviceManagerCallback (
 ;

 

 /**

-  EDES_TODO: Add function description

+  EDES_TODO: Add function description.

 

   @param VOID            EDES_TODO: Add parameter description

 

@@ -111,7 +111,7 @@ InitializeDeviceManager (
 ;

 

 /**

-  EDES_TODO: Add function description

+  EDES_TODO: Add function description.

 

   @param VOID            EDES_TODO: Add parameter description