Reviewed-by: Yonghong Zhu <[email protected]> Best Regards, Zhu Yonghong
-----Original Message----- From: edk2-devel [mailto:[email protected]] On Behalf Of Jaben Carsey Sent: Thursday, March 29, 2018 8:02 AM To: [email protected] Subject: [edk2] [PATCH v1 0/4] BaseTools: eliminate some redundant code. no external behavorial change, but remove some confusing and unneeded code. Jaben Carsey (4): BaseTools: no need to do int() API work for it BaseTools: use in to compare single chars BaseTools: remove loop and variables. BaseTools: cleanup class heirarchy BaseTools/Source/Python/AutoGen/GenC.py | 5 +---- BaseTools/Source/Python/Common/DecClassObject.py | 13 +------------ BaseTools/Source/Python/Common/DscClassObject.py | 13 +------------ BaseTools/Source/Python/Common/Expression.py | 6 +++--- BaseTools/Source/Python/Common/FdfClassObject.py | 12 +----------- BaseTools/Source/Python/Common/InfClassObject.py | 13 +------------ BaseTools/Source/Python/Common/Misc.py | 13 ------------- BaseTools/Source/Python/Ecc/MetaFileWorkspace/MetaFileParser.py | 13 +------------ BaseTools/Source/Python/GenFds/FdfParser.py | 5 +---- BaseTools/Source/Python/PatchPcdValue/PatchPcdValue.py | 10 ++-------- 10 files changed, 12 insertions(+), 91 deletions(-) -- 2.16.2.windows.1 _______________________________________________ edk2-devel mailing list [email protected] https://lists.01.org/mailman/listinfo/edk2-devel _______________________________________________ edk2-devel mailing list [email protected] https://lists.01.org/mailman/listinfo/edk2-devel
