On 2023-10-25 15:22, Nicola Vetrini wrote:
The clauses of a switch should be enclosed directly by a switch
statement to make the code more easily understandable and less
prone to errors.
Signed-off-by: Nicola Vetrini <nicola.vetr...@bugseng.com>
---
This patch is mainly indended to probe how the community, especially
the
maintainers, would receive such modifications to the code, and whether
there
would be consensus on the rule's adoption. Anyone is welcome to
give feedback on this, especially on the x86 side, where this pattern
is used more frequently.
---
xen/arch/x86/hvm/vlapic.c | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
Cc: all the relevant maintainers (mostly x86)
An update on the status of this rule's adoption. It has been deemed not
beneficial to go after violations in xen/arch/x86/x86_emulate/.* ,
therefore the only patches to be expected for this rule will be about
the few remaining violations in both arm, x86 and common code.
--
Nicola Vetrini, BSc
Software Engineer, BUGSENG srl (https://bugseng.com)
