Re: [PATCH 0/7] Fix MISRA C 2012 Rule 20.7 violations

2022-09-29 Thread Xenia Ragiadakou
Hi Roberto, On 9/28/22 17:11, Roberto Bagnara wrote: On 9/26/22 10:50, Xenia Ragiadakou wrote: On 9/18/22 16:02, Roberto Bagnara wrote: The question is on the interpretation of Rule 20.7. Are parenthesis required by Rule 20.7 in the following cases: - macro parameters used as function argumen

Re: [PATCH 0/7] Fix MISRA C 2012 Rule 20.7 violations

2022-09-28 Thread Roberto Bagnara
Hi Xenia. Please see below. On 9/26/22 10:50, Xenia Ragiadakou wrote: On 9/18/22 16:02, Roberto Bagnara wrote: The question is on the interpretation of Rule 20.7. Are parenthesis required by Rule 20.7 in the following cases: - macro parameters used as function arguments  > [...]  > - macro p

Re: [PATCH 0/7] Fix MISRA C 2012 Rule 20.7 violations

2022-09-26 Thread Xenia Ragiadakou
Hi Roberto, On 9/18/22 16:02, Roberto Bagnara wrote: On 03/09/22 02:52, Stefano Stabellini wrote: +Roberto I think we need Roberto's advice on Rule 20.7. (Full thread below.) Hi there, sorry for the delay: I missed this message. Please see below, where I took the freedom of rearranging the c

Re: [PATCH 0/7] Fix MISRA C 2012 Rule 20.7 violations

2022-09-19 Thread Jan Beulich
On 18.09.2022 15:02, Roberto Bagnara wrote: > On 03/09/22 02:52, Stefano Stabellini wrote: >> The question is on the interpretation of Rule 20.7. Are parenthesis >> required by Rule 20.7 in the following cases: >> >> - macro parameters used as function arguments > > [...] > > - macro parameter us

Re: [PATCH 0/7] Fix MISRA C 2012 Rule 20.7 violations

2022-09-18 Thread Roberto Bagnara
On 03/09/22 02:52, Stefano Stabellini wrote: +Roberto I think we need Roberto's advice on Rule 20.7. (Full thread below.) Hi there, sorry for the delay: I missed this message. Please see below, where I took the freedom of rearranging the cases. The question is on the interpretation of Rule 2

Re: [PATCH 0/7] Fix MISRA C 2012 Rule 20.7 violations

2022-09-02 Thread Stefano Stabellini
+Roberto I think we need Roberto's advice on Rule 20.7. (Full thread below.) The question is on the interpretation of Rule 20.7. Are parenthesis required by Rule 20.7 in the following cases: - macro parameters used as function arguments - macro parameters used as macro arguments - macro paramet

Re: [PATCH 0/7] Fix MISRA C 2012 Rule 20.7 violations

2022-09-02 Thread Xenia Ragiadakou
On 9/2/22 05:07, Stefano Stabellini wrote: On Thu, 1 Sep 2022, Bertrand Marquis wrote: Hi Xenia, On 1 Sep 2022, at 10:27, Xenia Ragiadakou wrote: On 9/1/22 01:35, Stefano Stabellini wrote: Patches 1, 4, and 6 are already committed. I plan to commit patches 2, 3 and 5 in the next couple o

Re: [PATCH 0/7] Fix MISRA C 2012 Rule 20.7 violations

2022-09-01 Thread Stefano Stabellini
On Thu, 1 Sep 2022, Bertrand Marquis wrote: > Hi Xenia, > > > On 1 Sep 2022, at 10:27, Xenia Ragiadakou wrote: > > > > > > On 9/1/22 01:35, Stefano Stabellini wrote: > >> Patches 1, 4, and 6 are already committed. I plan to commit patches 2, 3 > >> and 5 in the next couple of days. > >> Patch 7

Re: [PATCH 0/7] Fix MISRA C 2012 Rule 20.7 violations

2022-09-01 Thread Bertrand Marquis
Hi Xenia, > On 1 Sep 2022, at 10:27, Xenia Ragiadakou wrote: > > > On 9/1/22 01:35, Stefano Stabellini wrote: >> Patches 1, 4, and 6 are already committed. I plan to commit patches 2, 3 >> and 5 in the next couple of days. >> Patch 7 needs further discussions and it is best addressed during the

Re: [PATCH 0/7] Fix MISRA C 2012 Rule 20.7 violations

2022-09-01 Thread Xenia Ragiadakou
On 9/1/22 01:35, Stefano Stabellini wrote: Patches 1, 4, and 6 are already committed. I plan to commit patches 2, 3 and 5 in the next couple of days. Patch 7 needs further discussions and it is best addressed during the next MISRA C sync-up. I would like to share here, before the next MISRA

Re: [PATCH 0/7] Fix MISRA C 2012 Rule 20.7 violations

2022-08-31 Thread Stefano Stabellini
Patches 1, 4, and 6 are already committed. I plan to commit patches 2, 3 and 5 in the next couple of days. Patch 7 needs further discussions and it is best addressed during the next MISRA C sync-up. On Fri, 19 Aug 2022, Xenia Ragiadakou wrote: > Xenia Ragiadakou (7): > xen/arm: gic_v3_its: Fix

[PATCH 0/7] Fix MISRA C 2012 Rule 20.7 violations

2022-08-19 Thread Xenia Ragiadakou
Xenia Ragiadakou (7): xen/arm: gic_v3_its: Fix MISRA C 2012 Rule 20.7 violations xsm/flask: sidtab: Fix MISRA C 2012 Rule 20.7 violations xen/elf: Fix MISRA C 2012 Rule 20.7 violations xen/vgic: Fix MISRA C 2012 Rule 20.7 violation xen/rbtree: Fix MISRA C 2012 Rule 20.7 violation xen/ar