diff options
author | Gary Lin <glin@suse.com> | 2018-06-25 18:31:35 +0800 |
---|---|---|
committer | Yonghong Zhu <yonghong.zhu@intel.com> | 2018-06-27 16:33:27 +0800 |
commit | 0d1f5b2b5dc3c1cf381be0a1ec8f960dc6029a93 (patch) | |
tree | 370c6121707d6141988e1795f061e93abc112cbb /BaseTools/Source/Python/TargetTool | |
parent | 2617a73c3628473bacea887d885bdd1e7808ccc6 (diff) | |
download | edk2-0d1f5b2b5dc3c1cf381be0a1ec8f960dc6029a93.tar.gz edk2-0d1f5b2b5dc3c1cf381be0a1ec8f960dc6029a93.tar.bz2 edk2-0d1f5b2b5dc3c1cf381be0a1ec8f960dc6029a93.zip |
BaseTools: Fix old python2 idioms
Based on "futurize -f lib2to3.fixes.fix_idioms"
* Change some type comparisons to isinstance() calls:
type(x) == T -> isinstance(x, T)
type(x) is T -> isinstance(x, T)
type(x) != T -> not isinstance(x, T)
type(x) is not T -> not isinstance(x, T)
* Change "while 1:" into "while True:".
* Change both
v = list(EXPR)
v.sort()
foo(v)
and the more general
v = EXPR
v.sort()
foo(v)
into
v = sorted(EXPR)
foo(v)
Contributed-under: TianoCore Contribution Agreement 1.1
Cc: Yonghong Zhu <yonghong.zhu@intel.com>
Cc: Liming Gao <liming.gao@intel.com>
Signed-off-by: Gary Lin <glin@suse.com>
Reviewed-by: Yonghong Zhu <yonghong.zhu@intel.com>
Diffstat (limited to 'BaseTools/Source/Python/TargetTool')
-rw-r--r-- | BaseTools/Source/Python/TargetTool/TargetTool.py | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/BaseTools/Source/Python/TargetTool/TargetTool.py b/BaseTools/Source/Python/TargetTool/TargetTool.py index ed567b8708..26d2bb9ebf 100644 --- a/BaseTools/Source/Python/TargetTool/TargetTool.py +++ b/BaseTools/Source/Python/TargetTool/TargetTool.py @@ -83,7 +83,7 @@ class TargetTool(): def Print(self):
errMsg = ''
for Key in self.TargetTxtDictionary:
- if type(self.TargetTxtDictionary[Key]) == type([]):
+ if isinstance(self.TargetTxtDictionary[Key], type([])):
print("%-30s = %s" % (Key, ''.join(elem + ' ' for elem in self.TargetTxtDictionary[Key])))
elif self.TargetTxtDictionary[Key] is None:
errMsg += " Missing %s configuration information, please use TargetTool to set value!" % Key + os.linesep
|