08-02-2009, 08:35 AM
What do people think of comments like these in code
/* defines to satisfy the MISRA checker tool */
... inevitable strange pre-processor magic
#if defined ( MISRA_CHECK )
.. behaviour 1
#endif
I see worrying amounts of it in a project I am working with and it makes me think I am working with unfinished code where the developers would rather continue writing insecure code by keeping a checker quiet than addressing the root causes. That this has been allowed by the checking tool vendor without
Does this kind of thing satisfy the spirit of MISRA and should checking tool vendors be inventing their own suppression language to litter the C-code with?
Does it break rule 2.4 - sections of code should not be commented out
Could we have a rule that bans checking tool logic, defines and comments in production code. Could we add a rule that requires checking tool vendors be required to discard the content of all well formed comments and not add mechansims for the programmer to determine if a checking tool is parsing the code? please.
Jonathan
/* defines to satisfy the MISRA checker tool */
... inevitable strange pre-processor magic
#if defined ( MISRA_CHECK )
.. behaviour 1
#endif
I see worrying amounts of it in a project I am working with and it makes me think I am working with unfinished code where the developers would rather continue writing insecure code by keeping a checker quiet than addressing the root causes. That this has been allowed by the checking tool vendor without
Does this kind of thing satisfy the spirit of MISRA and should checking tool vendors be inventing their own suppression language to litter the C-code with?
Does it break rule 2.4 - sections of code should not be commented out
Could we have a rule that bans checking tool logic, defines and comments in production code. Could we add a rule that requires checking tool vendors be required to discard the content of all well formed comments and not add mechansims for the programmer to determine if a checking tool is parsing the code? please.
Jonathan
<t></t>