Re: Experiments with clang-format and exclude-list.json

[Stefano Stabellini <sstabellini@xxxxxxxxxx>]

On Tue, 18 Jul 2023, Jan Beulich wrote:
> On 18.07.2023 11:13, Luca Fancellu wrote:
> > I’ve done some experiment with clang-format and the hypervisor code 
> > (xen/xen/**), I had the need to exclude some files from the tool
> > and I thought to use the exclude-list.json, but since it was intended for 
> > MISRA, I’m now asking your feedback about that.
> > 
> > In my opinion it could make sense, because with that list we are excluding 
> > everything that is external and not adopting the Xen
> > coding style.
> > 
> > Do you have an opinion on that? Can it be one file? Or should it be a 
> > separate file only for the coding style exclusion?
> 
> It feels like back when the file was introduced I already commented on
> the potential for it to be multi-use, so long as it's actually dealing
> with the slightly separate needs. IOW I think it can be one file, but
> it needs to allow for files that want excluding from rule checkers, but
> not from style ones, and vice versa. Whether specific "comment" wording
> would be suitable, or whether instead separate attributes would want
> introducing is TBD.

I agree