On 16/01/24 14:02, Jan Beulich wrote:
On 16.01.2024 12:58, Andrew Cooper wrote:
Fixes MISRA XXX
Rule 5.5 if I'm not mistaken; had to look it up for the patch sent
earlier in the day. As to "fixes" - when it's not an actual bug, I had
(successfully) asked the bugseng guys to avoid that term, and instead
use "addresses" or "eliminates a ... violation" or some such.
No functional change.
Signed-off-by: Andrew Cooper <[email protected]>
Reviewed-by: Jan Beulich <[email protected]>
Jan
I confirm that it is Rule 5.5.
I would like to point out that although the patch fixes violations of
Rule 5.5, it introduces new violations of Rule 5.3 "An identifier
declared in an inner scope shall not hide an identifier declared in an
outer scope": cpuid is used also as an identifier for some formal
arguments (the pipeline does not fail because Rule 5.3 is not tagged
as "clean" and the introduction of new violations does not cause
a failure).
A solution could be to rename the function adding a prefix or a suffix
to its name.
--
Federico Serafini, M.Sc.
Software Engineer, BUGSENG (http://bugseng.com)
