On Fri, Jul 19, 2019 at 2:43 AM Julien Grall <julien.gr...@arm.com> wrote: > > Hi Tamas, > > On 18/07/2019 18:48, Tamas K Lengyel wrote: > >> - Line 1025: The tools needs to be able to deal for_each_vcpu(...) & > >> co. > > > > These can be made OK by adding braces. Other than that the only way I > > found to make it not change the indentation is to add the comment "/* > > *INDENT-OFF* */" before the block and "/* *INDENT-ON* */" afterwards. > > None of them looks really appealing because it means astyle will not correctly > indent if the user does not add braces or comments. > > Could astyle be easily modified to recognize foreach macros?
Not that I'm aware of. If you don't want to manually annotate files with unsupported macros then just exclude those files from astyle. I wouldn't recommend adding this to the CI for all files, only for those that their respective maintainers have confirmed to conform to the style and want to enforce it going forward. Tamas _______________________________________________ Xen-devel mailing list Xen-devel@lists.xenproject.org https://lists.xenproject.org/mailman/listinfo/xen-devel
