From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mga11.intel.com (mga11.intel.com [192.55.52.93]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by ml01.01.org (Postfix) with ESMTPS id 8DB1B81EE6 for ; Thu, 24 Nov 2016 17:20:51 -0800 (PST) Received: from orsmga003.jf.intel.com ([10.7.209.27]) by fmsmga102.fm.intel.com with ESMTP; 24 Nov 2016 17:20:50 -0800 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.31,692,1473145200"; d="scan'208";a="905239024" Received: from shwdeopenpsi114.ccr.corp.intel.com ([10.239.157.135]) by orsmga003.jf.intel.com with ESMTP; 24 Nov 2016 17:20:49 -0800 From: Dandan Bi To: edk2-devel@lists.01.org Cc: Pete Batard , Jiewen Yao Date: Fri, 25 Nov 2016 09:19:40 +0800 Message-Id: <1480036780-88716-1-git-send-email-dandan.bi@intel.com> X-Mailer: git-send-email 1.9.5.msysgit.1 Subject: [patch] MdeModulePkg: Fix GCC build failure X-BeenThere: edk2-devel@lists.01.org X-Mailman-Version: 2.1.21 Precedence: list List-Id: EDK II Development List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Fri, 25 Nov 2016 01:20:51 -0000 Remove the variables which are set but not used. Cc: Pete Batard Cc: Jiewen Yao Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi --- MdeModulePkg/Universal/EbcDxe/EbcDebugger/Edb.c | 3 --- MdeModulePkg/Universal/EbcDxe/EbcDebugger/EdbCmdBreakpoint.c | 5 ----- MdeModulePkg/Universal/EbcDxe/EbcDebugger/EdbHook.c | 9 ++++----- MdeModulePkg/Universal/EbcDxe/EbcDebugger/EdbSymbol.c | 2 -- 4 files changed, 4 insertions(+), 15 deletions(-) diff --git a/MdeModulePkg/Universal/EbcDxe/EbcDebugger/Edb.c b/MdeModulePkg/Universal/EbcDxe/EbcDebugger/Edb.c index e0b85c7..adc1c50 100644 --- a/MdeModulePkg/Universal/EbcDxe/EbcDebugger/Edb.c +++ b/MdeModulePkg/Universal/EbcDxe/EbcDebugger/Edb.c @@ -192,11 +192,10 @@ Returns: --*/ { UINT64 Address; UINTN Index; - UINT16 OldInstruction; BOOLEAN IsHitBreakpoint; // // Roll back IP for breakpoint instruction (BREAK(3) : 0x0300) // @@ -207,11 +206,10 @@ Returns: // IsHitBreakpoint = FALSE; for (Index = 0; (Index < DebuggerPrivate->DebuggerBreakpointCount) && (Index < EFI_DEBUGGER_BREAKPOINT_MAX); Index++) { if ((DebuggerPrivate->DebuggerBreakpointContext[Index].BreakpointAddress == Address) && (DebuggerPrivate->DebuggerBreakpointContext[Index].State)) { - OldInstruction = (UINT16)DebuggerPrivate->DebuggerBreakpointContext[Index].OldInstruction; IsHitBreakpoint = TRUE; break; } } @@ -242,11 +240,10 @@ Returns: // IsHitBreakpoint = FALSE; for (Index = 0; (Index < DebuggerPrivate->DebuggerBreakpointCount) && (Index < EFI_DEBUGGER_BREAKPOINT_MAX); Index++) { if ((DebuggerPrivate->DebuggerBreakpointContext[Index].BreakpointAddress == Address) && (DebuggerPrivate->DebuggerBreakpointContext[Index].State)) { - OldInstruction = (UINT16)DebuggerPrivate->DebuggerBreakpointContext[Index].OldInstruction; IsHitBreakpoint = TRUE; break; } } diff --git a/MdeModulePkg/Universal/EbcDxe/EbcDebugger/EdbCmdBreakpoint.c b/MdeModulePkg/Universal/EbcDxe/EbcDebugger/EdbCmdBreakpoint.c index 4be6dd4..a137ea9 100644 --- a/MdeModulePkg/Universal/EbcDxe/EbcDebugger/EdbCmdBreakpoint.c +++ b/MdeModulePkg/Universal/EbcDxe/EbcDebugger/EdbCmdBreakpoint.c @@ -437,12 +437,10 @@ Returns: --*/ { UINTN Index; EFI_STATUS Status; - UINTN Address; - UINT16 OldInstruction; if (CommandArg == NULL) { EDBPrint (L"BreakpointClear Argument error!\n"); return EFI_DEBUG_CONTINUE; } @@ -464,13 +462,10 @@ Returns: if ((Index >= EFI_DEBUGGER_BREAKPOINT_MAX) || (Index >= DebuggerPrivate->DebuggerBreakpointCount)) { EDBPrint (L"BreakpointClear error!\n"); return EFI_DEBUG_CONTINUE; - } else { - Address = (UINTN)DebuggerPrivate->DebuggerBreakpointContext[Index].BreakpointAddress; - OldInstruction = (UINT16)DebuggerPrivate->DebuggerBreakpointContext[Index].OldInstruction; } // // Delete breakpoint // diff --git a/MdeModulePkg/Universal/EbcDxe/EbcDebugger/EdbHook.c b/MdeModulePkg/Universal/EbcDxe/EbcDebugger/EdbHook.c index 9196adb..696ab74 100644 --- a/MdeModulePkg/Universal/EbcDxe/EbcDebugger/EdbHook.c +++ b/MdeModulePkg/Universal/EbcDxe/EbcDebugger/EdbHook.c @@ -387,11 +387,10 @@ Arguments: Returns: None --*/ { - EFI_STATUS Status; UINTN Index; EFI_DEBUGGER_SYMBOL_OBJECT *Object; EFI_DEBUGGER_SYMBOL_ENTRY *Entry; @@ -431,35 +430,35 @@ Returns: } // // locate PciRootBridgeIo // - Status = gBS->LocateProtocol ( + gBS->LocateProtocol ( &gEfiPciRootBridgeIoProtocolGuid, NULL, (VOID**) &mDebuggerPrivate.PciRootBridgeIo ); // // locate DebugImageInfoTable // - Status = EfiGetSystemConfigurationTable ( + EfiGetSystemConfigurationTable ( &gEfiDebugImageInfoTableGuid, (VOID**) &mDebuggerPrivate.DebugImageInfoTableHeader ); // // Create break event // - Status = gBS->CreateEvent ( + gBS->CreateEvent ( EVT_TIMER | EVT_NOTIFY_SIGNAL, TPL_CALLBACK, EbcDebuggerBreakEventFunc, NULL, &mDebuggerPrivate.BreakEvent ); - Status = gBS->SetTimer ( + gBS->SetTimer ( mDebuggerPrivate.BreakEvent, TimerPeriodic, EFI_DEBUG_BREAK_TIMER_INTERVAL ); diff --git a/MdeModulePkg/Universal/EbcDxe/EbcDebugger/EdbSymbol.c b/MdeModulePkg/Universal/EbcDxe/EbcDebugger/EdbSymbol.c index 9573c43..a1d21aa 100644 --- a/MdeModulePkg/Universal/EbcDxe/EbcDebugger/EdbSymbol.c +++ b/MdeModulePkg/Universal/EbcDxe/EbcDebugger/EdbSymbol.c @@ -191,19 +191,17 @@ Returns: --*/ { CHAR8 *LineBuffer; CHAR8 *FieldBuffer; - EFI_DEBUGGER_SYMBOL_ENTRY *Entry; EDB_EBC_MAP_PARSE_STATE MapParseState; EDB_EBC_SYMBOL_PARSE_STATE SymbolParseState; CHAR8 *Name; CHAR8 *ObjName; UINTN Address; EFI_DEBUGGER_SYMBOL_TYPE Type; - Entry = Object->Entry; // // Begin to parse the Buffer // LineBuffer = AsciiStrGetNewTokenLine (Buffer, "\n\r"); -- 1.9.5.msysgit.1