clang can format the macros just fine as they are. I added clang for checking the style of c++ code (mainly vapi code). C files are still checked using indent, which doesn't understand the macros ... Also I wasn't able to come up with a clang-format which matches the current indent style 1:1 so I don't see a switch to clang-format coming any time soon...
Thanks, Klement Quoting Stephen Hemminger (2018-11-16 02:02:55) > I just discovered the clang-format file and noticed that Linux kernel used > the ForEachMacros option to format for() wrappers. > > VPP seems to be riddled with INDENT-OFF when it does vector loops. (2123 > times!) > Has anyone looked into replacing these with ForEachMacros? > > > -=-=-=-=-=-=-=-=-=-=-=- > Links: You receive all messages sent to this group. > > View/Reply Online (#11274): https://lists.fd.io/g/vpp-dev/message/11274 > Mute This Topic: https://lists.fd.io/mt/28156993/675704 > Group Owner: vpp-dev+ow...@lists.fd.io > Unsubscribe: https://lists.fd.io/g/vpp-dev/unsub [ksek...@cisco.com] > -=-=-=-=-=-=-=-=-=-=-=-
-=-=-=-=-=-=-=-=-=-=-=- Links: You receive all messages sent to this group. View/Reply Online (#11280): https://lists.fd.io/g/vpp-dev/message/11280 Mute This Topic: https://lists.fd.io/mt/28156993/21656 Group Owner: vpp-dev+ow...@lists.fd.io Unsubscribe: https://lists.fd.io/g/vpp-dev/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-
