Re: [edk2-devel] [PATCH v1 1/1] BaseTools/Ecc: Make Ecc only check first include guard

2021-03-14 Thread Bob Feng 
This patch looks good to me.
After you remove the duplicated arm copyright, Reviewed-by: Bob Feng 


-Original Message-
From: devel@edk2.groups.io  On Behalf Of PierreGondois
Sent: Wednesday, March 10, 2021 12:17 AM
To: devel@edk2.groups.io; Feng, Bob C ; 
gaolim...@byosoft.com.cn; Chen, Christine 
Subject: [edk2-devel] [PATCH v1 1/1] BaseTools/Ecc: Make Ecc only check first 
include guard

From: Pierre Gondois 

The Ecc tool checks the format of the include guard. This check is currently 
done on all the names following the '#ifndef' statement.
It should only be done on the first include guard.

REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3252
Signed-off-by: Pierre Gondois 
---
The changes can be seen at: 
https://github.com/PierreARM/edk2/tree/1640_Ecc_tool_corrections

 BaseTools/Source/Python/Ecc/Check.py | 9 ++---
 1 file changed, 6 insertions(+), 3 deletions(-)

diff --git a/BaseTools/Source/Python/Ecc/Check.py 
b/BaseTools/Source/Python/Ecc/Check.py
index 7a012617fd35..d82b42de0119 100644
--- a/BaseTools/Source/Python/Ecc/Check.py
+++ b/BaseTools/Source/Python/Ecc/Check.py
@@ -3,6 +3,7 @@
 #
 # Copyright (c) 2021, Arm Limited. All rights reserved.  # Copyright (c) 
2008 - 2020, Intel Corporation. All rights reserved.
+# Copyright (c) 2021, Arm Limited. All rights reserved.
 # SPDX-License-Identifier: BSD-2-Clause-Patent  #  from __future__ import 
absolute_import @@ -1437,11 +1438,13 @@ class Check(object):

 SqlCommand = """select ID, Value from %s where Model = %s""" % 
(FileTable, MODEL_IDENTIFIER_MACRO_IFNDEF)
 RecordSet = EccGlobalData.gDb.TblFile.Exec(SqlCommand)
-for Record in RecordSet:
-Name = Record[1].replace('#ifndef', '').strip()
+if RecordSet:
+# Only check the first ifndef statement of the file
+FirstDefine = sorted(RecordSet, key=lambda Record: 
Record[0])[0]
+Name = FirstDefine[1].replace('#ifndef', '').strip()
 if Name[0] == '_' or Name[-1] != '_' or Name[-2] == '_':
 if not 
EccGlobalData.gException.IsException(ERROR_NAMING_CONVENTION_CHECK_IFNDEF_STATEMENT,
 Name):
-
EccGlobalData.gDb.TblReport.Insert(ERROR_NAMING_CONVENTION_CHECK_IFNDEF_STATEMENT,
 OtherMsg="The #ifndef name [%s] does not follow the rules" % (Name), 
BelongsToTable=FileTable, BelongsToItem=Record[0])
+
+ EccGlobalData.gDb.TblReport.Insert(ERROR_NAMING_CONVENTION_CHECK_IFNDE
+ F_STATEMENT, OtherMsg="The #ifndef name [%s] does not follow the 
+ rules" % (Name), BelongsToTable=FileTable, 
+ BelongsToItem=FirstDefine[0])

 # Rule for path name, variable name and function name
 # 1. First character should be upper case
--
2.17.1








-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#72751): https://edk2.groups.io/g/devel/message/72751
Mute This Topic: https://groups.io/mt/81204854/21656
Group Owner: devel+ow...@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-

回复: [edk2-devel] 回复: [edk2-devel] [PATCH v1 1/1] BaseTools/Ecc: Make Ecc only check first include guard

2021-03-14 Thread gaoliming 
Pierre:

 I see. This change is good to me. Reviewed-by: Liming Gao 


 

Thanks

Liming

发件人: PierreGondois  
发送时间: 2021年3月11日 17:26
收件人: gaoliming ; devel@edk2.groups.io
主题: Re: [edk2-devel] 回复: [edk2-devel] [PATCH v1 1/1] BaseTools/Ecc: Make Ecc 
only check first include guard

 

Hello Liming,
Yes this check is only done for '.h' files. Cf the bit of code from the same 
file:

if os.path.splitext(F)[1] in ('.h'):
  self.NamingConventionCheckIfndefStatement(FileTable)

I think this is normal, the format of the initial 'ifndef' needs to be 
formatted as 'Name_', but not necessarilly all the other 'ifndef' in the file.
I added the arm copyright two times, I will submit a v2 if the patch goes 
forward.

Regards,
Pierre 



-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#72750): https://edk2.groups.io/g/devel/message/72750
Mute This Topic: https://groups.io/mt/81340012/21656
Group Owner: devel+ow...@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-

Re: [edk2-devel] 回复: [edk2-devel] [PATCH v1 1/1] BaseTools/Ecc: Make Ecc only check first include guard

2021-03-11 Thread PierreGondois 
Hello Liming,
Yes this check is only done for '.h' files. Cf the bit of code from the same 
file:

if os.path.splitext(F)[1] in ('.h'):
self.NamingConventionCheckIfndefStatement(FileTable)

I think this is normal, the format of the initial 'ifndef' needs to be 
formatted as 'Name_', but not necessarilly all the other 'ifndef' in the file.
I added the arm copyright two times, I will submit a v2 if the patch goes 
forward.

Regards,
Pierre


-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#72664): https://edk2.groups.io/g/devel/message/72664
Mute This Topic: https://groups.io/mt/81243933/21656
Group Owner: devel+ow...@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-

回复: [edk2-devel] [PATCH v1 1/1] BaseTools/Ecc: Make Ecc only check first include guard

2021-03-10 Thread gaoliming 
Is this checker for .h file only?

> -邮件原件-
> 发件人: devel@edk2.groups.io  代表
> PierreGondois
> 发送时间: 2021年3月10日 0:17
> 收件人: devel@edk2.groups.io; bob.c.f...@intel.com;
> gaolim...@byosoft.com.cn; yuwei.c...@intel.com
> 主题: [edk2-devel] [PATCH v1 1/1] BaseTools/Ecc: Make Ecc only check first
> include guard
> 
> From: Pierre Gondois 
> 
> The Ecc tool checks the format of the include guard. This check is
> currently done on all the names following the '#ifndef' statement.
> It should only be done on the first include guard.
> 
> REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3252
> Signed-off-by: Pierre Gondois 
> ---
> The changes can be seen at:
> https://github.com/PierreARM/edk2/tree/1640_Ecc_tool_corrections
> 
>  BaseTools/Source/Python/Ecc/Check.py | 9 ++---
>  1 file changed, 6 insertions(+), 3 deletions(-)
> 
> diff --git a/BaseTools/Source/Python/Ecc/Check.py
> b/BaseTools/Source/Python/Ecc/Check.py
> index 7a012617fd35..d82b42de0119 100644
> --- a/BaseTools/Source/Python/Ecc/Check.py
> +++ b/BaseTools/Source/Python/Ecc/Check.py
> @@ -3,6 +3,7 @@
>  #
>  # Copyright (c) 2021, Arm Limited. All rights reserved.
>  # Copyright (c) 2008 - 2020, Intel Corporation. All rights reserved.
> +# Copyright (c) 2021, Arm Limited. All rights reserved.
>  # SPDX-License-Identifier: BSD-2-Clause-Patent
>  #
>  from __future__ import absolute_import
> @@ -1437,11 +1438,13 @@ class Check(object):
> 
>  SqlCommand = """select ID, Value from %s where Model
> = %s""" % (FileTable, MODEL_IDENTIFIER_MACRO_IFNDEF)
>  RecordSet = EccGlobalData.gDb.TblFile.Exec(SqlCommand)
> -for Record in RecordSet:
> -Name = Record[1].replace('#ifndef', '').strip()
> +if RecordSet:
> +# Only check the first ifndef statement of the file
> +FirstDefine = sorted(RecordSet, key=lambda Record:
> Record[0])[0]
> +Name = FirstDefine[1].replace('#ifndef', '').strip()
>  if Name[0] == '_' or Name[-1] != '_' or Name[-2] == '_':
>  if not
> EccGlobalData.gException.IsException(ERROR_NAMING_CONVENTION_CHE
> CK_IFNDEF_STATEMENT, Name):
> -
> EccGlobalData.gDb.TblReport.Insert(ERROR_NAMING_CONVENTION_CHECK
> _IFNDEF_STATEMENT, OtherMsg="The #ifndef name [%s] does not follow the
> rules" % (Name), BelongsToTable=FileTable, BelongsToItem=Record[0])
> +
> EccGlobalData.gDb.TblReport.Insert(ERROR_NAMING_CONVENTION_CHECK
> _IFNDEF_STATEMENT, OtherMsg="The #ifndef name [%s] does not follow the
> rules" % (Name), BelongsToTable=FileTable, BelongsToItem=FirstDefine[0])
> 
>  # Rule for path name, variable name and function name
>  # 1. First character should be upper case
> --
> 2.17.1
> 
> 
> 
> 
> 





-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#72655): https://edk2.groups.io/g/devel/message/72655
Mute This Topic: https://groups.io/mt/81243933/21656
Group Owner: devel+ow...@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-

[edk2-devel] [PATCH v1 1/1] BaseTools/Ecc: Make Ecc only check first include guard

2021-03-09 Thread PierreGondois 
From: Pierre Gondois 

The Ecc tool checks the format of the include guard. This check is
currently done on all the names following the '#ifndef' statement.
It should only be done on the first include guard.

REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3252
Signed-off-by: Pierre Gondois 
---
The changes can be seen at: 
https://github.com/PierreARM/edk2/tree/1640_Ecc_tool_corrections

 BaseTools/Source/Python/Ecc/Check.py | 9 ++---
 1 file changed, 6 insertions(+), 3 deletions(-)

diff --git a/BaseTools/Source/Python/Ecc/Check.py 
b/BaseTools/Source/Python/Ecc/Check.py
index 7a012617fd35..d82b42de0119 100644
--- a/BaseTools/Source/Python/Ecc/Check.py
+++ b/BaseTools/Source/Python/Ecc/Check.py
@@ -3,6 +3,7 @@
 #
 # Copyright (c) 2021, Arm Limited. All rights reserved.
 # Copyright (c) 2008 - 2020, Intel Corporation. All rights reserved.
+# Copyright (c) 2021, Arm Limited. All rights reserved.
 # SPDX-License-Identifier: BSD-2-Clause-Patent
 #
 from __future__ import absolute_import
@@ -1437,11 +1438,13 @@ class Check(object):

 SqlCommand = """select ID, Value from %s where Model = %s""" % 
(FileTable, MODEL_IDENTIFIER_MACRO_IFNDEF)
 RecordSet = EccGlobalData.gDb.TblFile.Exec(SqlCommand)
-for Record in RecordSet:
-Name = Record[1].replace('#ifndef', '').strip()
+if RecordSet:
+# Only check the first ifndef statement of the file
+FirstDefine = sorted(RecordSet, key=lambda Record: 
Record[0])[0]
+Name = FirstDefine[1].replace('#ifndef', '').strip()
 if Name[0] == '_' or Name[-1] != '_' or Name[-2] == '_':
 if not 
EccGlobalData.gException.IsException(ERROR_NAMING_CONVENTION_CHECK_IFNDEF_STATEMENT,
 Name):
-
EccGlobalData.gDb.TblReport.Insert(ERROR_NAMING_CONVENTION_CHECK_IFNDEF_STATEMENT,
 OtherMsg="The #ifndef name [%s] does not follow the rules" % (Name), 
BelongsToTable=FileTable, BelongsToItem=Record[0])
+
EccGlobalData.gDb.TblReport.Insert(ERROR_NAMING_CONVENTION_CHECK_IFNDEF_STATEMENT,
 OtherMsg="The #ifndef name [%s] does not follow the rules" % (Name), 
BelongsToTable=FileTable, BelongsToItem=FirstDefine[0])

 # Rule for path name, variable name and function name
 # 1. First character should be upper case
--
2.17.1



-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#72595): https://edk2.groups.io/g/devel/message/72595
Mute This Topic: https://groups.io/mt/81204854/21656
Group Owner: devel+ow...@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-