From nobody Mon Feb 9 19:59:32 2026 Delivered-To: importer@patchew.org Authentication-Results: mx.zohomail.com; spf=none (zoho.com: 198.145.21.10 is neither permitted nor denied by domain of lists.01.org) smtp.mailfrom=edk2-devel-bounces@lists.01.org Return-Path: Received: from ml01.01.org (ml01.01.org [198.145.21.10]) by mx.zohomail.com with SMTPS id 1522970074633702.2054480186297; Thu, 5 Apr 2018 16:14:34 -0700 (PDT) Received: from [127.0.0.1] (localhost [IPv6:::1]) by ml01.01.org (Postfix) with ESMTP id BF2862265A8C6; Thu, 5 Apr 2018 16:14:10 -0700 (PDT) Received: from mga07.intel.com (mga07.intel.com [134.134.136.100]) (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 9C9CA225264AB for ; Thu, 5 Apr 2018 16:14:07 -0700 (PDT) Received: from orsmga004.jf.intel.com ([10.7.209.38]) by orsmga105.jf.intel.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 05 Apr 2018 16:14:06 -0700 Received: from jcarsey-desk1.amr.corp.intel.com ([10.7.159.141]) by orsmga004.jf.intel.com with ESMTP; 05 Apr 2018 16:14:05 -0700 X-Original-To: edk2-devel@lists.01.org Received-SPF: none (zoho.com: 198.145.21.10 is neither permitted nor denied by domain of lists.01.org) client-ip=198.145.21.10; envelope-from=edk2-devel-bounces@lists.01.org; helo=ml01.01.org; Received-SPF: Pass (sender SPF authorized) identity=mailfrom; client-ip=134.134.136.100; helo=mga07.intel.com; envelope-from=jaben.carsey@intel.com; receiver=edk2-devel@lists.01.org X-Amp-Result: SKIPPED(no attachment in message) X-Amp-File-Uploaded: False X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.48,412,1517904000"; d="scan'208";a="189047119" From: Jaben Carsey To: edk2-devel@lists.01.org Date: Thu, 5 Apr 2018 16:14:02 -0700 Message-Id: <14593a5f937eed96986ff2f01ade60055b9d79f0.1522969978.git.jaben.carsey@intel.com> X-Mailer: git-send-email 2.16.2.windows.1 In-Reply-To: References: In-Reply-To: References: Subject: [edk2] [PATCH v1 14/14] BaseTools: refactor and remove out of date use of .keys() X-BeenThere: edk2-devel@lists.01.org X-Mailman-Version: 2.1.26 Precedence: list List-Id: EDK II Development List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: Liming Gao MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable Errors-To: edk2-devel-bounces@lists.01.org Sender: "edk2-devel" X-ZohoMail: RSF_4 Z_629925259 SPT_0 Content-Type: text/plain; charset="utf-8" this is no longer required to make dictionary objects iterable. Cc: Liming Gao Cc: Yonghong Zhu Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Jaben Carsey --- BaseTools/Scripts/SmiHandlerProfileSymbolGen.py | 4 ++-- BaseTools/Source/Python/AutoGen/GenMake.py | 2 +- BaseTools/Source/Python/GenFds/Fv.py | 2 +- BaseTools/Source/Python/GenFds/Section.py | 5 +---- BaseTools/Source/Python/TargetTool/TargetTool.py | 3 +-- BaseTools/Source/Python/UPT/PomAdapter/InfPomAlignment.py | 5 +---- BaseTools/Source/Python/build/build.py | 4 +--- 7 files changed, 8 insertions(+), 17 deletions(-) diff --git a/BaseTools/Scripts/SmiHandlerProfileSymbolGen.py b/BaseTools/Sc= ripts/SmiHandlerProfileSymbolGen.py index f03278b64f8f..26c092410386 100644 --- a/BaseTools/Scripts/SmiHandlerProfileSymbolGen.py +++ b/BaseTools/Scripts/SmiHandlerProfileSymbolGen.py @@ -204,7 +204,7 @@ def genGuidString(guidreffile): if len(guidLineList) =3D=3D 2: guid =3D guidLineList[0] guidName =3D guidLineList[1] - if guid not in dictGuid.keys() : + if guid not in dictGuid : dictGuid[guid] =3D guidName =20 def createSym(symbolName): @@ -256,7 +256,7 @@ def main(): for smiEntry in SmiEntry: if smiEntry.hasAttribute("HandlerType"): guidValue =3D smiEntry.getAttribute("HandlerType") - if guidValue in dictGuid.keys() : + if guidValue in dictGuid: smiEntry.setAttribute("HandlerType", dictGuid[guidValu= e]) SmiHandler =3D smiEntry.getElementsByTagName("SmiHandler") for smiHandler in SmiHandler: diff --git a/BaseTools/Source/Python/AutoGen/GenMake.py b/BaseTools/Source/= Python/AutoGen/GenMake.py index dcdfcca1a5b0..61432516fd26 100644 --- a/BaseTools/Source/Python/AutoGen/GenMake.py +++ b/BaseTools/Source/Python/AutoGen/GenMake.py @@ -491,7 +491,7 @@ cleanlib: ImageEntryPoint =3D "_ModuleEntryPoint" =20 for k, v in self._AutoGenObject.Module.Defines.iteritems(): - if k not in self._AutoGenObject.Macros.keys(): + if k not in self._AutoGenObject.Macros: self._AutoGenObject.Macros[k] =3D v =20 if 'MODULE_ENTRY_POINT' not in self._AutoGenObject.Macros.keys(): diff --git a/BaseTools/Source/Python/GenFds/Fv.py b/BaseTools/Source/Python= /GenFds/Fv.py index 14e36b885966..345ad3bdcc90 100644 --- a/BaseTools/Source/Python/GenFds/Fv.py +++ b/BaseTools/Source/Python/GenFds/Fv.py @@ -318,7 +318,7 @@ class FV (FvClassObject): ' %s' %ErasePloarity + \ T_CHAR_LF) if not (self.FvAttributeDict is None): - for FvAttribute in self.FvAttributeDict.keys() : + for FvAttribute in self.FvAttributeDict: if FvAttribute =3D=3D "FvUsedSizeEnable": if self.FvAttributeDict[FvAttribute].upper() in ('TRUE= ', '1') : self.UsedSizeEnable =3D True diff --git a/BaseTools/Source/Python/GenFds/Section.py b/BaseTools/Source/P= ython/GenFds/Section.py index 5e0b4bee7d1c..6335c249a693 100644 --- a/BaseTools/Source/Python/GenFds/Section.py +++ b/BaseTools/Source/Python/GenFds/Section.py @@ -111,10 +111,7 @@ class Section (SectionClassObject): # @retval tuple (File list, boolean) # def GetFileList(FfsInf, FileType, FileExtension, Dict =3D {}, IsMakefi= le=3DFalse): - if FileType in Section.SectFileType.keys() : - IsSect =3D True - else : - IsSect =3D False + IsSect =3D FileType in Section.SectFileType =20 if FileExtension is not None: Suffix =3D FileExtension diff --git a/BaseTools/Source/Python/TargetTool/TargetTool.py b/BaseTools/S= ource/Python/TargetTool/TargetTool.py index ede9713c9b8b..143b53ec35ef 100644 --- a/BaseTools/Source/Python/TargetTool/TargetTool.py +++ b/BaseTools/Source/Python/TargetTool/TargetTool.py @@ -80,9 +80,8 @@ class TargetTool(): traceback.print_exception(last_type, last_value, last_tb) =20 def Print(self): - KeyList =3D self.TargetTxtDictionary.keys() errMsg =3D '' - for Key in KeyList: + for Key in self.TargetTxtDictionary: if type(self.TargetTxtDictionary[Key]) =3D=3D type([]): print "%-30s =3D %s" % (Key, ''.join(elem + ' ' for elem i= n self.TargetTxtDictionary[Key])) elif self.TargetTxtDictionary[Key] is None: diff --git a/BaseTools/Source/Python/UPT/PomAdapter/InfPomAlignment.py b/Ba= seTools/Source/Python/UPT/PomAdapter/InfPomAlignment.py index e37a0b6c3be7..165ac111272a 100644 --- a/BaseTools/Source/Python/UPT/PomAdapter/InfPomAlignment.py +++ b/BaseTools/Source/Python/UPT/PomAdapter/InfPomAlignment.py @@ -486,10 +486,7 @@ class InfPomAlignment(ModuleObject): # # Get all LibraryClasses # - LibClassObj =3D self.Parser.InfLibraryClassSection.LibraryClas= ses - Keys =3D LibClassObj.keys() - for Key in Keys: - LibraryClassData =3D LibClassObj[Key] + for LibraryClassData in self.Parser.InfLibraryClassSection.Lib= raryClasses.values(): for Item in LibraryClassData: LibraryClass =3D CommonObject.LibraryClassObject() LibraryClass.SetUsage(DT.USAGE_ITEM_CONSUMES) diff --git a/BaseTools/Source/Python/build/build.py b/BaseTools/Source/Pyth= on/build/build.py index f211f8c64116..ceacc124933c 100644 --- a/BaseTools/Source/Python/build/build.py +++ b/BaseTools/Source/Python/build/build.py @@ -1421,9 +1421,7 @@ class Build(): def _RebaseModule (self, MapBuffer, BaseAddress, ModuleList, AddrIsOff= set =3D True, ModeIsSmm =3D False): if ModeIsSmm: AddrIsOffset =3D False - InfFileNameList =3D ModuleList.keys() - #InfFileNameList.sort() - for InfFile in InfFileNameList: + for InfFile in ModuleList: sys.stdout.write (".") sys.stdout.flush() ModuleInfo =3D ModuleList[InfFile] --=20 2.16.2.windows.1 _______________________________________________ edk2-devel mailing list edk2-devel@lists.01.org https://lists.01.org/mailman/listinfo/edk2-devel