From: Jaben Carsey <jaben.carsey@intel.com>
To: edk2-devel@lists.01.org
Subject: [PATCH v1 0/9] BaseTools: refactor Workspace classes
Date: Wed, 29 Aug 2018 08:45:35 -0700 [thread overview]
Message-ID: <cover.1535557474.git.jaben.carsey@intel.com> (raw)
update the classes for the following:
1) use decorators for property
2) use decorators for caching property and caching function
- this allows for objects to reduce in size as they get used
3) remove unused variables and properties
4) use tuple instead of custom class when apropriate
5) remove callers from accessing "private" data and use the existing
properties
6) removed a circular dependency between APIs
Jaben Carsey (9):
BaseTools: Refactor PlatformAutoGen
BaseTools: AutoGen refactor WorkspaceAutoGen class
BaseTools: AutoGen - refactor class properties
BaseTools: refactor class properties
BaseTools: Workspace classes refactor properties
BaseTools: refactor Build Database objects
BaseTools: Don't save unused workspace data
BaseTools: refactor to not overcreate ModuleAutoGen objects
BaseTools: refactor to cache InfBuildData data
BaseTools/Source/Python/AutoGen/AutoGen.py | 691 +++++++-------
BaseTools/Source/Python/AutoGen/GenMake.py | 20 +-
BaseTools/Source/Python/Common/Misc.py | 90 +-
BaseTools/Source/Python/GenFds/FfsInfStatement.py | 4 +-
BaseTools/Source/Python/Workspace/BuildClassObject.py | 39 +-
BaseTools/Source/Python/Workspace/DecBuildData.py | 65 +-
BaseTools/Source/Python/Workspace/DscBuildData.py | 151 +--
BaseTools/Source/Python/Workspace/InfBuildData.py | 965 +++++++++-----------
BaseTools/Source/Python/Workspace/MetaFileParser.py | 18 +-
BaseTools/Source/Python/Workspace/WorkspaceDatabase.py | 16 +-
BaseTools/Source/Python/build/build.py | 4 +-
11 files changed, 938 insertions(+), 1125 deletions(-)
--
2.16.2.windows.1
next reply other threads:[~2018-08-29 15:50 UTC|newest]
Thread overview: 10+ messages / expand[flat|nested] mbox.gz Atom feed top
2018-08-29 15:45 Jaben Carsey [this message]
2018-08-29 15:45 ` [PATCH v1 1/9] BaseTools: Refactor PlatformAutoGen Jaben Carsey
2018-08-29 15:45 ` [PATCH v1 2/9] BaseTools: AutoGen refactor WorkspaceAutoGen class Jaben Carsey
2018-08-29 15:45 ` [PATCH v1 3/9] BaseTools: AutoGen - refactor class properties Jaben Carsey
2018-08-29 15:45 ` [PATCH v1 4/9] BaseTools: " Jaben Carsey
2018-08-29 15:45 ` [PATCH v1 5/9] BaseTools: Workspace classes refactor properties Jaben Carsey
2018-08-29 15:45 ` [PATCH v1 6/9] BaseTools: refactor Build Database objects Jaben Carsey
2018-08-29 15:45 ` [PATCH v1 7/9] BaseTools: Don't save unused workspace data Jaben Carsey
2018-08-29 15:45 ` [PATCH v1 8/9] BaseTools: refactor to not overcreate ModuleAutoGen objects Jaben Carsey
2018-08-29 15:45 ` [PATCH v1 9/9] BaseTools: refactor to cache InfBuildData data Jaben Carsey
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.1535557474.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