On 2024-11-19 00:54, Stefano Stabellini wrote:
Currently the identifiers characters limit is arbitrarily set to 40. It
causes a few violations as we have some identifiers longer than 40.
Increase the limit to another rather arbitrary limit of 63. Thanks to
this change, we remove a few violations, getting us one step closer to
marking Rules 5.2 and 5.4 as clean.
The ECLAIR configuration is already using 63, so this change matches
the rules.rst documentation with the ECLAIR behavior.
Signed-off-by: Stefano Stabellini <stefano.stabell...@amd.com>
Reviewed-by: Nicola Vetrini <nicola.vetr...@bugseng.com>
---
Changes in v2:
- use 63 instead of 64
- drop unneeded change to ECLAIR config
docs/misra/rules.rst | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/docs/misra/rules.rst b/docs/misra/rules.rst
index 4a144da8d6..3ed5801bff 100644
--- a/docs/misra/rules.rst
+++ b/docs/misra/rules.rst
@@ -154,7 +154,7 @@ maintainers if you want to suggest a change.
* - `Rule 5.1
<https://gitlab.com/MISRA/MISRA-C/MISRA-C-2012/Example-Suite/-/blob/master/R_05_01_2.c>`_
- Required
- External identifiers shall be distinct
- - The Xen characters limit for identifiers is 40. Public headers
+ - The Xen characters limit for identifiers is 63. Public headers
(xen/include/public/) are allowed to retain longer identifiers
for backward compatibility.
@@ -162,7 +162,7 @@ maintainers if you want to suggest a change.
- Required
- Identifiers declared in the same scope and name space shall be
distinct
- - The Xen characters limit for identifiers is 40. Public headers
+ - The Xen characters limit for identifiers is 63. Public headers
(xen/include/public/) are allowed to retain longer identifiers
for backward compatibility.
@@ -177,7 +177,7 @@ maintainers if you want to suggest a change.
* - `Rule 5.4
<https://gitlab.com/MISRA/MISRA-C/MISRA-C-2012/Example-Suite/-/blob/master/R_05_04.c>`_
- Required
- Macro identifiers shall be distinct
- - The Xen characters limit for macro identifiers is 40. Public
+ - The Xen characters limit for macro identifiers is 63. Public
headers (xen/include/public/) are allowed to retain longer
identifiers for backward compatibility.
--
Nicola Vetrini, BSc
Software Engineer, BUGSENG srl (https://bugseng.com)
