diff options
author | Fan, ZhijuX <zhijux.fan@intel.com> | 2019-09-04 17:54:12 +0800 |
---|---|---|
committer | Feng, Bob C <bob.c.feng@intel.com> | 2019-09-09 11:14:42 +0800 |
commit | d6f5a5050b942cc0ec08bdc39900b8231957e18c (patch) | |
tree | 8257ebe476c560c270eec7f0a7c7daf6369ea720 /BaseTools/Source/Python/Ecc/Check.py | |
parent | 03ad3d889c64978c31a8ccd70f1862558bdb0a2d (diff) | |
download | edk2-d6f5a5050b942cc0ec08bdc39900b8231957e18c.tar.gz edk2-d6f5a5050b942cc0ec08bdc39900b8231957e18c.tar.bz2 edk2-d6f5a5050b942cc0ec08bdc39900b8231957e18c.zip |
BaseTools:ECC need to handle lower case 'static'
BZ:https://bugzilla.tianocore.org/show_bug.cgi?id=1778
We are going to add keyword (lower case) 'static' for functions and global
variables which are not referenced outside their current C file.
However, the ECC tool only recognizes upper case 'STATIC' at this moment.
This will lead to issue reports for new codes that follow the above coding
style.
This patch is going to handle lower case 'static'
Cc: Liming Gao <liming.gao@intel.com>
Cc: Bob Feng <bob.c.feng@intel.com>
Signed-off-by: Zhiju.Fan <zhijux.fan@intel.com>
Reviewed-by: Bob Feng <bob.c.feng@intel.com>
Diffstat (limited to 'BaseTools/Source/Python/Ecc/Check.py')
-rw-r--r-- | BaseTools/Source/Python/Ecc/Check.py | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/BaseTools/Source/Python/Ecc/Check.py b/BaseTools/Source/Python/Ecc/Check.py index 86bb8562ba..2180818609 100644 --- a/BaseTools/Source/Python/Ecc/Check.py +++ b/BaseTools/Source/Python/Ecc/Check.py @@ -441,7 +441,7 @@ class Check(object): self.DeclCheckUnionType()
- # Check whether no use of int, unsigned, char, void, static, long in any .c, .h or .asl files.
+ # Check whether no use of int, unsigned, char, void, long in any .c, .h or .asl files.
def DeclCheckNoUseCType(self):
if EccGlobalData.gConfig.DeclarationDataTypeCheckNoUseCType == '1' or EccGlobalData.gConfig.DeclarationDataTypeCheckAll == '1' or EccGlobalData.gConfig.CheckAll == '1':
EdkLogger.quiet("Checking Declaration No use C type ...")
|