diff options
author | qwang12 <qwang12@6f19259b-4bc3-4df7-8a09-765794883524> | 2008-07-07 09:17:56 +0000 |
---|---|---|
committer | qwang12 <qwang12@6f19259b-4bc3-4df7-8a09-765794883524> | 2008-07-07 09:17:56 +0000 |
commit | a78b08d13b86c86dcfbe1c9e88a66b2b18ee7575 (patch) | |
tree | 9c2d898860f7a253495aaed4e9432277863ecaf0 /MdeModulePkg/Universal/BdsDxe | |
parent | f3947f1ac735011e482b5aab452f843ddca6cfe1 (diff) | |
download | edk2-platforms-a78b08d13b86c86dcfbe1c9e88a66b2b18ee7575.tar.xz |
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
Diffstat (limited to 'MdeModulePkg/Universal/BdsDxe')
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
|