[edk2] [PATCH v1 0/4] BaseTools: eliminate some redundant code.

Jaben Carsey posted 4 patches 6 years ago
Failed in applying to current master (apply log)
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(-)
[edk2] [PATCH v1 0/4] BaseTools: eliminate some redundant code.
Posted by Jaben Carsey 6 years ago
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
edk2-devel@lists.01.org
https://lists.01.org/mailman/listinfo/edk2-devel
Re: [edk2] [PATCH v1 0/4] BaseTools: eliminate some redundant code.
Posted by Zhu, Yonghong 6 years ago
Reviewed-by: Yonghong Zhu <yonghong.zhu@intel.com> 

Best Regards,
Zhu Yonghong


-----Original Message-----
From: edk2-devel [mailto:edk2-devel-bounces@lists.01.org] On Behalf Of Jaben Carsey
Sent: Thursday, March 29, 2018 8:02 AM
To: edk2-devel@lists.01.org
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
edk2-devel@lists.01.org
https://lists.01.org/mailman/listinfo/edk2-devel
_______________________________________________
edk2-devel mailing list
edk2-devel@lists.01.org
https://lists.01.org/mailman/listinfo/edk2-devel