diff options
Diffstat (limited to 'MdeModulePkg/Universal')
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
|