public inbox for devel@edk2.groups.io
 help / color / mirror / Atom feed
From: Jaben <jaben.carsey@intel.com>
To: edk2-devel@lists.01.org
Subject: [PATCH v1 0/4] BaseTools: refactoring (and one bug fix)
Date: Wed, 11 Apr 2018 16:08:04 -0700	[thread overview]
Message-ID: <cover.1523487973.git.jaben.carsey@intel.com> (raw)

refactor to reduce duplication
remove a duplicate function name (I presume this branch never executes
as it would have had exception)

Jaben (4):
  BaseTools: remove duplicate function name
  BaseTools: replace 'UINT8','UINT16','UINT32','UINT64','VOID*' with
    shared constants.
  BaseTools: Remove un-needed list comprehension
  BaseTools: merge towards minimum PCD MAX <something> methods

 BaseTools/Source/Python/AutoGen/AutoGen.py                      | 26 +++---
 BaseTools/Source/Python/AutoGen/GenC.py                         | 94 ++++++++++----------
 BaseTools/Source/Python/AutoGen/GenPcdDb.py                     | 23 ++---
 BaseTools/Source/Python/AutoGen/GenVar.py                       | 29 +++---
 BaseTools/Source/Python/AutoGen/ValidCheckingInfoObject.py      |  9 +-
 BaseTools/Source/Python/Common/DataType.py                      | 16 ++--
 BaseTools/Source/Python/Common/Expression.py                    | 61 ++++++-------
 BaseTools/Source/Python/Common/Misc.py                          | 12 +--
 BaseTools/Source/Python/Common/RangeExpression.py               | 25 ++----
 BaseTools/Source/Python/Common/VpdInfoFile.py                   |  9 +-
 BaseTools/Source/Python/Ecc/MetaFileWorkspace/MetaFileParser.py |  2 +-
 BaseTools/Source/Python/Eot/FvImage.py                          |  4 +-
 BaseTools/Source/Python/GenFds/FdfParser.py                     | 15 ++--
 BaseTools/Source/Python/GenFds/FfsInfStatement.py               | 11 ++-
 BaseTools/Source/Python/GenFds/GenFdsGlobalVariable.py          |  4 +-
 BaseTools/Source/Python/PatchPcdValue/PatchPcdValue.py          | 21 ++---
 BaseTools/Source/Python/Workspace/DscBuildData.py               | 69 ++++++--------
 BaseTools/Source/Python/build/BuildReport.py                    |  4 +-
 18 files changed, 211 insertions(+), 223 deletions(-)

-- 
2.16.2.windows.1



             reply	other threads:[~2018-04-11 23:08 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2018-04-11 23:08 Jaben [this message]
2018-04-11 23:08 ` [PATCH v1 1/4] BaseTools: remove duplicate function name Jaben
2018-04-18  8:37   ` Zhu, Yonghong
2018-04-11 23:08 ` [PATCH v1 2/4] BaseTools: replace 'UINT8', 'UINT16', 'UINT32', 'UINT64', 'VOID*' with shared constants Jaben
2018-04-17 13:32   ` Zhu, Yonghong
2018-04-11 23:08 ` [PATCH v1 3/4] BaseTools: Remove un-needed list comprehension Jaben
2018-04-18  8:38   ` Zhu, Yonghong
2018-04-11 23:08 ` [PATCH v1 4/4] BaseTools: merge towards minimum PCD MAX <something> methods Jaben
2018-04-12  0:40   ` Feng, Bob C

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-list from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=cover.1523487973.git.jaben.carsey@intel.com \
    --to=devel@edk2.groups.io \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox