> On 9 Aug 2023, at 16:41, Jan Beulich <jbeul...@suse.com> wrote:
>
> On 28.07.2023 10:11, Luca Fancellu wrote:
>> This script finds every .c and .h file in the xen hypervisor
>> codebase, takes the exclusion list from docs/misra, removes the
>> file excluded from the list and for the remaining files is
>> calling clang-format on them.
>
> I was going to ask what the intended purpose is, but looking at the
> script itself I think I see the script can also be passed a set of
> file names, so it won't necessarily scan the entire code base.
> That'll make it more suitable to also be used in an incremental
> fashion (whether that's also useful to scan patches isn't really
> clear to me, but I guess that'll require yet more work).
Yes, when we will do some automation, it would be useful to just run the script
against the touched files from a patch.
Indeed running it against the patch file is very difficult, since we are doing
some
pre-process and post-process that requires some context that might be missing
from the code inside the patch.
Cheers,
Luca
