Fixes the GCC build 'Unused variable' warning (raised an error when warnings are treated as errors).

Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Olivier Martin <olivier.martin@arm.com>
Reviewed-by: Eric Dong <eric.dong@intel.com>

git-svn-id: https://svn.code.sf.net/p/edk2/code/trunk/edk2@14470 6f19259b-4bc3-4df7-8a09-765794883524

1 files changed, 24 insertions, 34 deletions

diff --git a/MdeModulePkg/Universal/HiiDatabaseDxe/ConfigRouting.c b/MdeModulePkg/Universal/HiiDatabaseDxe/ConfigRouting.c
index a83a2ff777..95ca8d4777 100644
--- a/MdeModulePkg/Universal/HiiDatabaseDxe/ConfigRouting.c
+++ b/MdeModulePkg/Universal/HiiDatabaseDxe/ConfigRouting.c
@@ -1430,26 +1430,18 @@ ParseIfrData (
   IFR_DEFAULT_DATA         DefaultData;

   IFR_DEFAULT_DATA         *DefaultDataPtr;

   IFR_BLOCK_DATA           *BlockData;

-  CHAR16                   *VarStoreName;

-  UINT16                   VarWidth;

-  UINT16                   VarDefaultId;

-  EFI_STRING               GuidStr;

-  EFI_STRING               NameStr;

-  EFI_STRING               TempStr;

-  UINTN                    LengthString;

-  BOOLEAN                  FirstOneOfOption;

-  LIST_ENTRY               *LinkData;

-  LIST_ENTRY               *LinkDefault;

-  EFI_IFR_VARSTORE_NAME_VALUE *IfrNameValueVarStore;

-

-  LengthString     = 0;

-  Status           = EFI_SUCCESS;

-  GuidStr          = NULL;

-  NameStr          = NULL;

-  TempStr          = NULL;

-  BlockData        = NULL;

-  DefaultDataPtr   = NULL;

-  FirstOneOfOption = FALSE;

+  CHAR16                   *VarStoreName;
+  UINT16                   VarWidth;
+  UINT16                   VarDefaultId;
+  BOOLEAN                  FirstOneOfOption;
+  LIST_ENTRY               *LinkData;
+  LIST_ENTRY               *LinkDefault;
+  EFI_IFR_VARSTORE_NAME_VALUE *IfrNameValueVarStore;
+
+  Status           = EFI_SUCCESS;
+  BlockData        = NULL;
+  DefaultDataPtr   = NULL;
+  FirstOneOfOption = FALSE;
   ZeroMem (&DefaultData, sizeof (IFR_DEFAULT_DATA));

 

   //

@@ -2872,13 +2864,12 @@ GetFullStringFromHiiFormPackages (
   IFR_DEFAULT_DATA             *DefaultIdArray;

   IFR_VARSTORAGE_DATA          *VarStorageData;

   EFI_STRING                   DefaultAltCfgResp;

-  EFI_STRING                   ConfigHdr;

-  EFI_STRING                   StringPtr;

-  EFI_STRING                   Progress;

-  BOOLEAN                      DataExist;

-

-  if (DataBaseRecord == NULL || DevicePath == NULL || Request == NULL || AltCfgResp == NULL) {

-    return EFI_INVALID_PARAMETER;

+  EFI_STRING                   ConfigHdr;
+  EFI_STRING                   StringPtr;
+  EFI_STRING                   Progress;
+
+  if (DataBaseRecord == NULL || DevicePath == NULL || Request == NULL || AltCfgResp == NULL) {
+    return EFI_INVALID_PARAMETER;
   }

 

   //

@@ -2888,13 +2879,12 @@ GetFullStringFromHiiFormPackages (
   DefaultIdArray    = NULL;

   VarStorageData    = NULL;

   DefaultAltCfgResp = NULL;

-  ConfigHdr         = NULL;

-  HiiFormPackage    = NULL;

-  PackageSize       = 0;

-  DataExist         = FALSE;

-  Progress          = *Request;

-

-  Status = GetFormPackageData (DataBaseRecord, &HiiFormPackage, &PackageSize);

+  ConfigHdr         = NULL;
+  HiiFormPackage    = NULL;
+  PackageSize       = 0;
+  Progress          = *Request;
+
+  Status = GetFormPackageData (DataBaseRecord, &HiiFormPackage, &PackageSize);
   if (EFI_ERROR (Status)) {

     return Status;

   }